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

Improvements to DID #85

Merged
merged 9 commits into from
Nov 23, 2022
Merged

Improvements to DID #85

merged 9 commits into from
Nov 23, 2022

Conversation

drbenvincent
Copy link
Collaborator

  • Partial progress on DiD: allow for multiple pre and post intervention observations #76. Makes DID analysis more robust. The initial implementation made too many assumptions making the code brittle and not generalisable to other datasets. It also assumed that there were multiple units per condition (ie multiple control and treatment units). This code is much more robust and now works when there is a single treatment group. That said, more work needs to be done in this case.
  • Adds a 'classic' example, the bank failure dataset, so contributes to Add examples for 'classic' causal inference datasets #44. So far this is only implemented with the Bayesian model, not the scikit-learn model.

@drbenvincent drbenvincent merged commit cce4909 into main Nov 23, 2022
@drbenvincent drbenvincent deleted the did-banks-example branch November 23, 2022 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant