-
Notifications
You must be signed in to change notification settings - Fork 5
feat: add Front-Door Criterion causal inference example (Uber–Lyft Tips) #36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
구현해주신 내용 보면서 많이 배웠습니다! 다만 전반적인 구조가 다소 복잡하기에 말씀하신 것처럼 중간중간에 DAG 시각화를 추가하고, 또 각 변수의 선정 이유, 맥락도 더 자세히 적어준다면 큰 도움이 될 것 같습니다! (예를 들어, M이 Mediator로 적용되는 이유, Mediator가 FrontDoor Criterion에 쓰이는 이유, 1단계와 2단계 추정치의 곱으로 추정하는 이유 등) ++) 문서의 목적을 생각하면 Illustrative 목적 보다는 실무 적용 사례에 가까운듯한데, SCM 안에 넣기보다는 별도의 실무 적용사례로 빼는 것도 괜찮을 것 같습니다! (해당 내용이 FrontDoor Criterion의 활용은 맞으나, 실무적인 반영요소 등으로 인해 해당 개념을 바로 이해하기는 힘들 것 같습니다.) 실제 데이터를 들고 오고, 또 결과도 해석할 가치가 크다는 점에서 많이 참고가 되었습니다. |
|
추가로 DoWhy기반 DAG 시각화 및 refute 과정도 같이 추가하여 이전 backdoor criterion과의 형식을 통일하면 좋을 것 같습니다! |
|
구현 내용
TODO
|
|
8주차_조해창님 코드리뷰.pdf |
|
해창님! 좋은 코드 구현덕에 많이 배웠습니다 ㅎㅎ |
| " graph=gml_frontdoor\n", | ||
| ")\n", | ||
| "\n", | ||
| "cm.view_model(layout=\"dot\")\n" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
제가 실행했을때 경로 없음으로 에러가 뜹니다 ㅜ
| "import duckdb, pandas as pd, numpy as np\n", | ||
| "import statsmodels.api as sm\n", | ||
| "import warnings\n", | ||
| "from dowhy import CausalModel\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dowhy패키지 설치 코드가 필요합니다!
구현 내용
방법
참고
TODO