Merge pull request #1821 from abhisrkckl/fdjumplog-y

Change default of FDJUMPLOG to Y

CHANGELOG-unreleased.md

@@ -11,6 +11,7 @@ the released changes.
 ### Changed
 - Moved the events -> TOAs and photon weights code into the function `load_events_weights` within `event_optimize`.
 - Updated the `maxMJD` argument in `event_optimize` to default to the current mjd
+- Changed default value of `FDJUMPLOG` to `Y`
 - Bumped `black` version to 24.x
 ### Added
 - Type hints in `pint.derived_quantities` and `pint.residuals`

src/pint/models/fdjump.py

@@ -69,7 +69,7 @@ def __init__(self):
         self.add_param(
             boolParameter(
                 name="FDJUMPLOG",
-                value=False,
+                value=True,
                 description="Whether to use log-frequency (Y) or linear-frequency (N) for computing FDJUMPs.",
             )
         )

tests/test_fdjump.py

@@ -41,6 +41,8 @@ def test_params(model_and_toas):
         and hasattr(model, "FDJUMPDM2")
     )
 
+    assert model.FDJUMPLOG.value
+
 
 def test_residuals(model_and_toas):
     model, toas = model_and_toas