Skip to content

Release v3.1.0#304

Merged
igerber merged 1 commit intomainfrom
release/v3.1.0
Apr 14, 2026
Merged

Release v3.1.0#304
igerber merged 1 commit intomainfrom
release/v3.1.0

Conversation

@igerber
Copy link
Copy Markdown
Owner

@igerber igerber commented Apr 14, 2026

Summary

  • Bump version to 3.1.0 across all locations (pyproject.toml, init.py, Cargo.toml, llms-full.txt, CITATION.cff)
  • Finalize CHANGELOG entry for v3.1.0

What's in v3.1.0

dCDH Phase 3 - complete feature set for ChaisemartinDHaultfoeuille:

The de Chaisemartin-D'Haultfœuille estimator now ships the full feature set for reversible-treatment DiD - the only such implementation across R, Stata, and Python that includes covariates, trends, and HonestDiD in a single package.

Methodology references (required if estimator / math changes)

Validation

Security / privacy

  • Confirm no secrets/PII in this PR: Yes

Generated with Claude Code

dCDH Phase 3 complete: covariates, group-specific linear trends,
HonestDiD sensitivity analysis, non-binary treatment support, and
placebo bootstrap SE. The de Chaisemartin-D'Haultfoeuille estimator
now ships the full feature set for reversible-treatment DiD.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

Overall Assessment

✅ Looks good — no unmitigated P0/P1 findings. The diff is release metadata/docs only; I found one P3 documentation overstatement in the changelog.

Executive Summary

Methodology

  • Severity: P3. Impact: CHANGELOG.md:14 says placebo SE via multiplier bootstrap “resolves Phase 1 deferral,” but the documented contract still says single-period DID_M^pl (L_max=None) has NaN SE, while only multi-horizon placebos (L_max >= 1) have analytical/bootstrap SE docs/methodology/REGISTRY.md:546-588. That limitation also remains tracked in TODO.md:59. Concrete fix: narrow the changelog wording to dynamic placebos / L_max >= 1, or explicitly note that single-period DID_M^pl SE remains deferred.
  • No other findings. Severity: none. Impact: no estimator math, weighting, variance, assumption checks, or defaults changed in this PR. Concrete fix: none.

Code Quality

Performance

  • No findings. Severity: none. Impact: no runtime paths or computational kernels changed. Concrete fix: none.

Maintainability

  • No findings. Severity: none. Impact: the diff is narrowly scoped and keeps release metadata aligned across ecosystems. Concrete fix: none.

Tech Debt

  • No findings. Severity: none. Impact: the underlying single-period placebo-SE limitation is already properly tracked in TODO.md:59, so it is not a blocker; only the new changelog wording needs alignment. Concrete fix: none beyond the P3 methodology note above.

Security

  • No findings. Severity: none. Impact: metadata/docs-only diff; no new secrets, inputs, or attack surface. Concrete fix: none.

Documentation/Tests

  • No additional findings beyond the changelog wording issue above. Severity: none. Impact: no executable logic changed, so the absence of test changes is acceptable for this release PR. Concrete fix: none.

@igerber igerber added the ready-for-ci Triggers CI test workflows label Apr 14, 2026
@igerber igerber merged commit 796ea14 into main Apr 14, 2026
12 of 25 checks passed
@igerber igerber deleted the release/v3.1.0 branch April 14, 2026 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-ci Triggers CI test workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant