Bring #177 up to date again (#198)#199
Closed
jorpppp wants to merge 2 commits into
Closed
Conversation
* #181 Fix check for constant treatment value and add returns (#182) * 186 error when normalizing 0 coefficient (#187) * #186 Fix bug when normalizing to 0 or positive * #186 Propagate the fix to _eventiv * PR for #170: let the window option choose a window range (#178) * #170 bring changes from main * #170 implement finding max or balanced window * #170 examples to test implementation * #170 add examples to test file * #170 change window description and add example in help file * #170 wording of calculated-window message * #170 modify description of left and right window * #170 wording of window limits * #170 wording of calculated-window message * #170 check for zero words in window * #170 rearrange parsewindow checks * #170 wording of window-parser error messges * #170 add to help file example about maximum window * #170 conflict with static * "#170 revert previous commit" This reverts commit c6d98aa. * #170 parse window only when it is non-empty * #170 window in cmdline parser * #170 Delete issue170 special directory * #170 wording of string window message * #170 let max or balanced only if impute is specified * #170 wording of error message * #170 restructure checks for staggered adoption * #170 restructure for IV case * #170 informative window message and error message for balanced * "#170 revert previous commit" This reverts commit fc197a9. * #170 informative window message and error message for balanced * #170 paragraph for max balanced * #170 check trend outside window for max and balanced * #170 remove comment * #170 trend specification if window is max or balanced * #170 clear message about treatment time * #170 check leads in proxyiv are in estimation window * #170 wording of message about calculated window * #178 Minor change to calculated window message * #170 mark non-missing obs in varlist * #170 examples to test implementation * #170 pass marker and revert syntax of eventgenvars * #170 Add marked sample to variables to keep when repeatedcs * #170 Delete issue170 directory * #178 Edits to helpfile --------- Co-authored-by: jorpppp <jorpppp@gmail.com> * PR for #179 Check behavior of the cohort and control_cohort options and allow automatic generation (#185) * #179 Create cohort_consistency.do * #179 Add consistency checks between cohort, control_cohort, and policyvar, add basic cohort variable creation * #179 Change cohort, control_cohort to string in xtevent * #179 Add bin and no reversion returns to _eventgenvars plus changes from #181 * #179 Enable saving created cohort variable * #179 Automatic generation of control_cohort * #179 Update help file * #179 Modify test file * #179 Delete issue folder * #179 #185 Generate cohort in sample that ignores missings in control vars * #179 #185 Add checks for existing cohort variables * #179 #185 SA and proxy not allowed * #179 #185 Fix bug in naming saved interaction variables * #179 #185 Fix sorting bug when creating cohorts in repeatedcs setting * PR for: #189 pre and post coefficients excluding omitted ones (#190) * #189 pre and post coefficients excluding omitted ones * #189 #190 enable diff avg in _eventiv * #189 #190 exclude endpoints from diffavg when trend is active --------- Co-authored-by: jorpppp <jorpppp@gmail.com> * 188 left window of zero returns an error (#194) * #188 ordering when there are not pre-event dummies * #188 ordering the left endpoint * #188 Do not test for pretrends for plot if left window is zero * #188 #194 Disable smoothest path if left window * PR for: #189 pre and post coefficients excluding omitted ones (#190) (#195) * #189 pre and post coefficients excluding omitted ones * #189 #190 enable diff avg in _eventiv * #189 #190 exclude endpoints from diffavg when trend is active --------- Co-authored-by: Constantino Carreto Romero <carretoromeroc@gmail.com> * #188 #194 Fix error messages for diffavg when windows are zero * #188 #194 Fix lead selection and normalization when left window is zero or when many coefs are normalized * #188 #194 Recheck that lead instrument is in estimation window when it is changed because normalizations coincide * #188 #194 Do not test linear pretrend if too few pre-event coefs --------- Co-authored-by: Constantino Carreto Romero <carretoromeroc@gmail.com> --------- Co-authored-by: Constantino Carreto Romero <carretoromeroc@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
impute(instag)only imputing missing policyvar values for cases when treatment changes within unit #181 Fix check for constant treatment value and add returns (#181 Fix check for constant treatment value and add returns #182)186 error when normalizing 0 coefficient (186 error when normalizing 0 coefficient #187)
Error when normalizing
0coefficient #186 Fix bug when normalizing to 0 or positiveError when normalizing
0coefficient #186 Propagate the fix to _eventivPR for Let the window option choose a window range based on event-time limits in the data #170: let the window option choose a window range (PR for #170: let the window option choose a window range #178)
Let the window option choose a window range based on event-time limits in the data #170 bring changes from main
Let the window option choose a window range based on event-time limits in the data #170 implement finding max or balanced window
Let the window option choose a window range based on event-time limits in the data #170 examples to test implementation
Let the window option choose a window range based on event-time limits in the data #170 add examples to test file
Let the window option choose a window range based on event-time limits in the data #170 change window description and add example in help file
Let the window option choose a window range based on event-time limits in the data #170 wording of calculated-window message
Let the window option choose a window range based on event-time limits in the data #170 modify description of left and right window
Let the window option choose a window range based on event-time limits in the data #170 wording of window limits
Let the window option choose a window range based on event-time limits in the data #170 wording of calculated-window message
Let the window option choose a window range based on event-time limits in the data #170 check for zero words in window
Let the window option choose a window range based on event-time limits in the data #170 rearrange parsewindow checks
Let the window option choose a window range based on event-time limits in the data #170 wording of window-parser error messges
Let the window option choose a window range based on event-time limits in the data #170 add to help file example about maximum window
Let the window option choose a window range based on event-time limits in the data #170 conflict with static
"Let the window option choose a window range based on event-time limits in the data #170 revert previous commit"
This reverts commit c6d98aa.
Let the window option choose a window range based on event-time limits in the data #170 parse window only when it is non-empty
Let the window option choose a window range based on event-time limits in the data #170 window in cmdline parser
Let the window option choose a window range based on event-time limits in the data #170 Delete issue170 special directory
Let the window option choose a window range based on event-time limits in the data #170 wording of string window message
Let the window option choose a window range based on event-time limits in the data #170 let max or balanced only if impute is specified
Let the window option choose a window range based on event-time limits in the data #170 wording of error message
Let the window option choose a window range based on event-time limits in the data #170 restructure checks for staggered adoption
Let the window option choose a window range based on event-time limits in the data #170 restructure for IV case
Let the window option choose a window range based on event-time limits in the data #170 informative window message and error message for balanced
"Let the window option choose a window range based on event-time limits in the data #170 revert previous commit"
This reverts commit fc197a9.
Let the window option choose a window range based on event-time limits in the data #170 informative window message and error message for balanced
Let the window option choose a window range based on event-time limits in the data #170 paragraph for max balanced
Let the window option choose a window range based on event-time limits in the data #170 check trend outside window for max and balanced
Let the window option choose a window range based on event-time limits in the data #170 remove comment
Let the window option choose a window range based on event-time limits in the data #170 trend specification if window is max or balanced
Let the window option choose a window range based on event-time limits in the data #170 clear message about treatment time
Let the window option choose a window range based on event-time limits in the data #170 check leads in proxyiv are in estimation window
Let the window option choose a window range based on event-time limits in the data #170 wording of message about calculated window
PR for #170: let the window option choose a window range #178 Minor change to calculated window message
Let the window option choose a window range based on event-time limits in the data #170 mark non-missing obs in varlist
Let the window option choose a window range based on event-time limits in the data #170 examples to test implementation
Let the window option choose a window range based on event-time limits in the data #170 pass marker and revert syntax of eventgenvars
Let the window option choose a window range based on event-time limits in the data #170 Add marked sample to variables to keep when repeatedcs
Let the window option choose a window range based on event-time limits in the data #170 Delete issue170 directory
PR for #170: let the window option choose a window range #178 Edits to helpfile
Co-authored-by: jorpppp jorpppp@gmail.com
PR for Check behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Check behavior of the cohort and control_cohort options and allow automatic generation (PR for #179 Check behavior of the cohort and control_cohort options and allow automatic generation #185)Check behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Create cohort_consistency.doCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Add consistency checks between cohort, control_cohort, and policyvar, add basic cohort variable creationCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Change cohort, control_cohort to string in xteventCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Add bin and no reversion returns to _eventgenvars plus changes fromimpute(instag)only imputing missing policyvar values for cases when treatment changes within unit #181Check behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Enable saving created cohort variableCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Automatic generation of control_cohortCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Update help fileCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Modify test fileCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 Delete issue folderCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 PR for #179 Check behavior of the cohort and control_cohort options and allow automatic generation #185 Generate cohort in sample that ignores missings in control varsCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 PR for #179 Check behavior of the cohort and control_cohort options and allow automatic generation #185 Add checks for existing cohort variablesCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 PR for #179 Check behavior of the cohort and control_cohort options and allow automatic generation #185 SA and proxy not allowedCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 PR for #179 Check behavior of the cohort and control_cohort options and allow automatic generation #185 Fix bug in naming saved interaction variablesCheck behavior of the
cohortandcontrol_cohortoptions and allow automatic generation #179 PR for #179 Check behavior of the cohort and control_cohort options and allow automatic generation #185 Fix sorting bug when creating cohorts in repeatedcs settingPR for: Difference of averages doesn't include all coefficients #189 pre and post coefficients excluding omitted ones (PR for: #189 pre and post coefficients excluding omitted ones #190)
Difference of averages doesn't include all coefficients #189 pre and post coefficients excluding omitted ones
Difference of averages doesn't include all coefficients #189 PR for: #189 pre and post coefficients excluding omitted ones #190 enable diff avg in _eventiv
Difference of averages doesn't include all coefficients #189 PR for: #189 pre and post coefficients excluding omitted ones #190 exclude endpoints from diffavg when trend is active
Co-authored-by: jorpppp jorpppp@gmail.com
188 left window of zero returns an error (188 left window of zero returns an error #194)
Left window of zero returns an error #188 ordering when there are not pre-event dummies
Left window of zero returns an error #188 ordering the left endpoint
Left window of zero returns an error #188 Do not test for pretrends for plot if left window is zero
Left window of zero returns an error #188 188 left window of zero returns an error #194 Disable smoothest path if left window
PR for: Difference of averages doesn't include all coefficients #189 pre and post coefficients excluding omitted ones (PR for: #189 pre and post coefficients excluding omitted ones #190) (PR for: #189 pre and post coefficients excluding omitted ones (#190) #195)
Difference of averages doesn't include all coefficients #189 pre and post coefficients excluding omitted ones
Difference of averages doesn't include all coefficients #189 PR for: #189 pre and post coefficients excluding omitted ones #190 enable diff avg in _eventiv
Difference of averages doesn't include all coefficients #189 PR for: #189 pre and post coefficients excluding omitted ones #190 exclude endpoints from diffavg when trend is active
Co-authored-by: Constantino Carreto Romero carretoromeroc@gmail.com
Left window of zero returns an error #188 188 left window of zero returns an error #194 Fix error messages for diffavg when windows are zero
Left window of zero returns an error #188 188 left window of zero returns an error #194 Fix lead selection and normalization when left window is zero or when many coefs are normalized
Left window of zero returns an error #188 188 left window of zero returns an error #194 Recheck that lead instrument is in estimation window when it is changed because normalizations coincide
Left window of zero returns an error #188 188 left window of zero returns an error #194 Do not test linear pretrend if too few pre-event coefs
Co-authored-by: Constantino Carreto Romero carretoromeroc@gmail.com
Co-authored-by: Constantino Carreto Romero carretoromeroc@gmail.com