Skip to content

Conversation

@igerber
Copy link
Owner

@igerber igerber commented Jan 25, 2026

Summary

  • Bump version to 2.1.7
  • Add CHANGELOG entry for v2.1.7 release

Changes in v2.1.7

Fixed

  • plot_event_study reference period normalization behavior
    • Effects are now only normalized when reference_period is explicitly provided
    • Auto-inferred reference periods only apply hollow marker styling (no normalization)
    • Reference period SE is set to NaN during normalization (constraint, not estimate)
    • Updated docstring to clarify explicit vs auto-inferred behavior

Changed

  • Refactored visualization tests to reuse cs_results fixture for better performance

Methodology references

Validation

Security / privacy

  • Confirm no secrets/PII in this PR: Yes

Generated with Claude Code

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions
Copy link

Overall Assessment

  • ✅ Looks good

Executive Summary

  • Metadata-only release: version bumps and changelog entry.
  • No estimator or inference logic changes; methodology registry checks not triggered.
  • No runtime code or tests changed in this PR.

Methodology

  • No findings.

Code Quality

  • No findings.

Performance

  • No findings.

Maintainability

  • No findings.

Tech Debt

  • No findings.

Security

  • No findings.

Documentation/Tests

  • No findings. Residual risk: metadata-only changes; no tests updated or required for this PR.

@igerber igerber merged commit da2d648 into main Jan 25, 2026
5 checks passed
@igerber igerber deleted the release/v2.1.7 branch January 25, 2026 11:59
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

Successfully merging this pull request may close these issues.

2 participants