Updated call to Pulsar in case of timing_package='pint'...

nanograv / enterprise

ENTERPRISE (Enhanced Numerical Toolbox Enabling a Robust PulsaR Inference SuitE) is a pulsar timing analysis code, aimed at noise analysis, gravitational-wave searches, and timing model analysis.

https://enterprise.readthedocs.io

MIT License

65 stars 67 forks source link

Updated call to Pulsar in case of timing_package='pint'... #240

Closed pennucci closed 3 years ago

pennucci commented 3 years ago

when using paths to par/tim files to use pint.models.get_model_and_toas.

Using get_model_and_toas is 'safer' in that it automatically applies commands from the timing model (e.g., clock, SS ephem) as necessary. This can still be overridden by the kwargs to Pulsar, but the default is to get them from the timing model.

pennucci commented 3 years ago

@Hazboun6 Can you review/merge this?

Hazboun6 commented 3 years ago

@pennucci This looks great. Thanks for bringing this to our attention and doing this so quickly!

Hazboun6 commented 3 years ago

@pennucci Sorry. I meant to merge ... Once the tests pass I'll merge.