Skip to content

Commit 2acf245

Browse files
authored
Merge pull request #207 from igerber/cs-edid-tests
Port R `did` package tests, fix not_yet_treated bugs
2 parents d7b394c + c8cd414 commit 2acf245

9 files changed

Lines changed: 2399 additions & 19 deletions

File tree

TODO.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -154,6 +154,18 @@ Spurious RuntimeWarnings ("divide by zero", "overflow", "invalid value") are emi
154154

155155
---
156156

157+
## Feature Gaps (from R `did` package comparison)
158+
159+
Features in R's `did` package that block porting additional tests:
160+
161+
| Feature | R tests blocked | Priority |
162+
|---------|----------------|----------|
163+
| Repeated cross-sections (`panel=FALSE`) | ~7 tests in test-att_gt.R + test-user_bug_fixes.R | Medium |
164+
| Sampling/population weights | 7 tests incl. all JEL replication | Medium |
165+
| Calendar time aggregation | 1 test in test-att_gt.R | Low |
166+
167+
---
168+
157169
## Performance Optimizations
158170

159171
Potential future optimizations:

0 commit comments

Comments
 (0)