Correct doc strings for Tempo par file reader and add additional parameters
All threads resolved!
Compare changes
Maintenance will be performed on git.ligo.org, containers.ligo.org, and docs.ligo.org on Tuesday 22 April 2025 starting at approximately 9am PDT. It is expected to take around 30 minutes and there will be several periods of downtime throughout the maintenance. Please address any comments, concerns, or questions to the helpdesk. This maintenance will be upgrading the GitLab database in order to be ready for the migration.
The doc strings for the parameters BETA
and LAMBDA
that can be read in from a Tempo-style parameter file are currently wrong - the doc strings states that these are Galactic latitude and longitude, when really they are just aliases for the ELAT
and ELONG
values, which are the ecliptic coordinates (see Table 2 of the Tempo2 manual). This MR corrects these doc strings.
In addition, it also adds in the ability to read in a couple of other potentially useful parameters from a par file (TIMEEPH
and T2CMETHOD
) and add checks to the test code to read in some additional parameters.
Please tick one of the following options:
For examples of changes that do not modify the API and/or are considered backwards (in)compatible, please see the contributing guide.