amend rng-med-win default for lalpulsar_ComputeFstatMCUpperLimit (!2450) · Merge requests · lscsoft / lalsuite · GitLab

Snippets Groups Projects

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.

Merged David Keitel requested to merge david-keitel/lalsuite:ComputeFstatMCUpperLimit_rng-med-win_default into master 3 weeks ago

Detailed Description

As per #792 (closed), change the local default of 50 to the shared standard provided by FstatOptionalArgsDefaults (currently 101).

API Changes

Please tick one of the following options:

These changes do not modify the API.
These changes do modify the API, and are backwards compatible.
These changes do modify the API, and are backwards incompatible.

For examples of changes that do not modify the API and/or are considered backwards (in)compatible, please see the contributing guide.

Justification for Backwards Incompatible Changes

Review Status

Activity

David Keitel added lalpulsar label 3 weeks ago

added lalpulsar label
David Keitel requested review from @karl-wette 3 weeks ago

requested review from @karl-wette
**** assigned to @david-keitel 3 weeks ago

assigned to @david-keitel
**** added apinone label 3 weeks ago

added apinone label
Karl Wette approved this merge request 2 weeks ago

approved this merge request
Karl Wette merged 2 weeks ago

merged
Karl Wette mentioned in commit 0901776d 2 weeks ago

mentioned in commit 0901776d

Please register or sign in to reply