Skip to content
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

regression adjusted estimator for propensity_score_weighting #407

Open
adg-ci opened this issue Apr 13, 2022 · 5 comments
Open

regression adjusted estimator for propensity_score_weighting #407

adg-ci opened this issue Apr 13, 2022 · 5 comments
Labels
good first issue Good for newcomers

Comments

@adg-ci
Copy link

adg-ci commented Apr 13, 2022

Thank you for developing this package.
I am looking for regression adjusted estimator for propensity_score_weighting.
Is that something the package would be supporting in the future?

Thanks.

@amit-sharma
Copy link
Member

That will be a very good addition. If I understand correctly, you are referring to the doubly robust estimator.

It may be a while before we get to it. Would you like to contribute and add it to the package?

@amit-sharma amit-sharma added the good first issue Good for newcomers label Apr 17, 2022
@adg-ci
Copy link
Author

adg-ci commented Apr 20, 2022

I am using this package for a project at work. I would have to see how the approval process works for such contributions. Not clear at the moment.

@emrekiciman
Copy link
Member

emrekiciman commented Apr 21, 2022 via email

@rahulbshrestha
Copy link
Contributor

rahulbshrestha commented May 4, 2024

Hi! I was thinking of working on this issue i.e implement the doubly robust estimator in doWhy. Do you have any pointers so I could get started?

@amit-sharma
Copy link
Member

Sure. At a high level, the doubly robust estimator is a combination of the regression estimator and the inverse propensity weighting estimator. So, you will need to create a new Estimator class having these models as part of it (e.g., as attributes).

For the exact way in which these two estimators should be combined, you can refer to, https://academic.oup.com/aje/article/173/7/761/103691. Let me know if you have any questions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants