Skip to content

Add idx_exclude to MarginalizingTimingModel - #450

Closed
vhaasteren wants to merge 101 commits into
nanograv:masterfrom
vhaasteren:idx_exclude
Closed

Add idx_exclude to MarginalizingTimingModel#450
vhaasteren wants to merge 101 commits into
nanograv:masterfrom
vhaasteren:idx_exclude

Conversation

@vhaasteren

Copy link
Copy Markdown
Member

Summary

  • Adds an idx_exclude parameter to MarginalizingTimingModel, mirroring the existing API on TimingModel.
  • Excluded columns are dropped before basis construction (including the SVD path), so the marginalized timing-model matrix only spans the remaining linear parameters.
  • Adds unit tests for default behavior, column exclusion, and SVD + exclusion, plus a likelihood smoke test.

This is a small API parity change. It lets callers marginalize over a subset of timing-model columns while handling the excluded ones elsewhere — for example, when a nonlinear delay term samples some parameters and the remaining linear timing parameters are marginalized analytically.

Test plan

  • pytest tests/test_gp_signals.py::TestGPSignals::test_marginalizing_timing_model_idx_exclude
  • pytest tests/test_likelihood.py::TestLikelihood::test_like_marginalizing_tm_idx_exclude
  • Full Enterprise test suite (if CI is available)

astrolamb and others added 27 commits February 12, 2026 20:58
Revert fastshermanmorrison-pulsar version
…ions

Update Python support to 3.10-3.14, drop EOL versions
…lized the duck-typing mechanism to check for callables
Update sqrtsolve to new fastshermanmorrison definitions
Fix: use duck-typing in Pulsar() to determine whether we have a libstempo or PINT object
Correcting "<" and ">" in description of parameters in `broken_powerlaw()`
Fix parameter descriptions in gp_priors.py
@vhaasteren vhaasteren self-assigned this May 29, 2026
@vhaasteren vhaasteren closed this May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants