You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi @apoorvalal , Rafi (@rafimikail) has signaled his interest to work on this. He'd pick up the work you started in #481. I guess you wouldn't insist on continuing to work on this? 😄
To get started, have you already taken a look at the vignette of the PanelView R package @rafimikail? This PR would mostly consist of adding the function arguments listed above as implemented in PanelView to Appoorva's PR and of course any other feature that you particularly like or see value in =) Is there anything you think we should be prioritizing @apoorvalal?
Context
It's often useful (and also recommended, see this tweet) to study the treatment rollout in DiD settings.
In the R world, the panelView package supports functionality to easily inspect treatment rollout in a panel context
Task
Enrich the implementation of the
panel_view
function in #481.It would be nice to add the following function arguments:
by_timing
, which sorts units by the timing of receiving the treatmentpre_post
leave_gap
collapse_history
Additionally, the function should throw a warning if there is a missing time variable for a given cohort.
The text was updated successfully, but these errors were encountered: