Skip to content

GitLab

Merged
Created by Matthew David Pitkin@matthew-pitkinDeveloper1 of 4 tasks completed1/4 tasks

PulsarParameterWrapper.py: fix outputting of glitch frequency second derivative output

Description

If you convert the pulsar parameters stored in a PulsarParametersPy structure to a string it correctly formats the outputs for all glitch parameters except the frequency second derivative offset. This MR fixes this, so that, e.g. rather than outputting

GLF20  # incorrect for the second derivative of the first glitch
GLF21  # incorrect for the second derivative of the second glitch

if correctly outputs

GLF2_0
GLF2_1

API Changes and Justification

Backwards Compatible Changes

  • This change introduces no API changes
  • This change adds new API calls

Backwards Incompatible Changes

  • This change modifies an existing API
  • This change removes an existing API

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

@karl-wette @david-keitel - this is a very minor change.