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

Handling of universal base period #34

Open
simonschoe opened this issue Jul 20, 2023 · 4 comments
Open

Handling of universal base period #34

simonschoe opened this issue Jul 20, 2023 · 4 comments

Comments

@simonschoe
Copy link

Hi there,

maybe the question is trivial but I couldn't really figure it out by reading the paper and README of the repo: How would I specify numPrePeriods and numPostPeriods in the call to createSensitivityResults_relativeMagnitudes when my base period is some pre-treatment period other the canoncial "-1"?

Let's say my event study estimates range from -8 to 8, with 0 being the first treatment period and having defined -4 as the reference period to which the estimates are normalized (due to anticipation effects). Would I then define numPrePeriods = 4 (-8,-7,-6,-5) or numPrePeriods = 7 (-8,-7,-6,-5,-3,-2,-1). I guess my issue here is that in the second case pre-treatment trends would be drawn from -5 to -3 right?

Grateful for any hints!

@jonathandroth
Copy link
Collaborator

jonathandroth commented Jul 21, 2023 via email

@jonathandroth
Copy link
Collaborator

@mcaceresb -- when you have RA time in the fall, one thing to put on your to-do list is to add a referencePeriod option (with default =-1), and to update the create_A_XX functions to impose base period of referencePeriod instead of -1. Thanks!

@simonschoe
Copy link
Author

Hi Jonathan, thanks for taking the time to reply, I appreciate it.

On the second point, HonestDID imposes bounds on the post-treatment
violations of parallel trends as a function of the pre-treatment
violations. For example, the "relative magnitudes" (RM) restriction with
M=1 says that the magnitude of the post-treatment violation is no larger
than the largest pre-treatment violation. But if you think that there's
anticipation in some pre-treatment periods, it is sort of weird to use
those periods for this calculation. Then you're saying that the
post-treatment bias is no larger than the sum of the pre-treatment
violations and pre-treatment anticipatory effects. Put otherwise, if you
think there's anticipation in the 3 periods before treatment, then you're
effectively saying that the start-date of the treatment effects is at -3.
So conceptually it might make more sense to use HonestDID only using the
"clean" pre-periods without anticipation.

So yes, thats exactly what I would like to do. I'd like to assess the sensitivity of the treatment effect estimate wrt violations in pre-treatment trends from -8 till -4, i.e., the pre-treatment period where we expect no anticipation.

Is modifying .create_A_RM the only feasible option here or could i also achieve the same thing by subsetting the betahat vector and sigma matrix that serve as inputs to createSensitivityResults_relativeMagnitudes?

@jonathandroth
Copy link
Collaborator

jonathandroth commented Jul 23, 2023 via email

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

No branches or pull requests

2 participants