Skip to content

Fix dollar sign escaping in continuous DiD tutorial#182

Merged
igerber merged 1 commit intomainfrom
fix/tutorial-dollar-sign-escape
Feb 22, 2026
Merged

Fix dollar sign escaping in continuous DiD tutorial#182
igerber merged 1 commit intomainfrom
fix/tutorial-dollar-sign-escape

Conversation

@igerber
Copy link
Owner

@igerber igerber commented Feb 22, 2026

Summary

  • Double-escape dollar signs (\\$) in tutorial 14 markdown cells so MathJax doesn't interpret paired $ on the same line as inline LaTeX delimiters
  • Fixes cells in Section 2 (Data Setup) and Section 4 (Interpreting dose-response curves)

Methodology references (required if estimator / math changes)

  • N/A — no methodology changes

Validation

  • Tests added/updated: No test changes (documentation-only fix)
  • Backtest / simulation / notebook evidence: Visual inspection of rendered markdown in Jupyter

Security / privacy

  • Confirm no secrets/PII in this PR: Yes

Generated with Claude Code

The previous single-backslash escape (\$) was not preventing MathJax
from interpreting paired dollar signs on the same line as inline LaTeX
delimiters. Double-escape (\\$) so markdown consumes one backslash
level and MathJax sees \$ as a literal dollar sign.

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

Overall assessment: ✅ Looks good

Executive Summary

  • Doc-only change: double-escaped dollar signs in tutorial markdown to avoid MathJax treating paired $ as inline delimiters. docs/tutorials/14_continuous_did.ipynb:L102-L109, docs/tutorials/14_continuous_did.ipynb:L298-L306
  • No estimator/assumption changes; tutorial content remains aligned with ContinuousDiD registry and estimator docstring. docs/methodology/REGISTRY.md:L398-L472, diff_diff/continuous_did.py:L1-L45
  • Edge-case checklist not triggered because no code paths changed.
  • Tests not run (documentation-only change).

Methodology

  • Severity: P3 | Impact: Escaping change only; ATT(d)/ACRT discussion unchanged and consistent with ContinuousDiD registry and docstring. | Fix: None. docs/tutorials/14_continuous_did.ipynb:L102-L109, docs/tutorials/14_continuous_did.ipynb:L298-L307, docs/methodology/REGISTRY.md:L398-L472, diff_diff/continuous_did.py:L1-L45

Code Quality

  • Severity: P3 | Impact: No code paths changed. | Fix: None.

Performance

  • Severity: P3 | Impact: No performance-related changes. | Fix: None.

Maintainability

  • Severity: P3 | Impact: No maintainability impact beyond documentation. | Fix: None.

Tech Debt

  • Severity: P3 | Impact: No tech-debt impact. | Fix: None.

Security

  • Severity: P3 | Impact: Doc-only edit; no runtime or secrets exposure. | Fix: None.

Documentation/Tests

  • Severity: P3 | Impact: Notebook markdown updated; tests not run. | Fix: Optionally re-render the notebook or docs build to verify $ displays correctly (no visible backslashes and no MathJax inline-math). docs/tutorials/14_continuous_did.ipynb:L102-L109, docs/tutorials/14_continuous_did.ipynb:L298-L306

@igerber igerber merged commit feaf199 into main Feb 22, 2026
1 check passed
@igerber igerber deleted the fix/tutorial-dollar-sign-escape branch February 22, 2026 21:21
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.

1 participant