Use double for cpnest parameter (!908) · Merge requests · lscsoft / bilby · GitLab

Snippets Groups Projects

Maintenance will be performed on git.ligo.org, containers.ligo.org, and docs.ligo.org on Tuesday 25 March 2025 starting at approximately 8:30am 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.

Merged Colm Talbot requested to merge fix-cpnest-cbc into master 4 years ago

The fix in !888 (merged) didn't actually fix the implementation for CBCs. The GPS times we use don't fit inside the float that was being used. This MR makes sure we use a double, which will hopefully be large enough for the foreseeable future.

Approval is optional

Activity

Colm Talbot added Bug Sampling labels 4 years ago

added Bug Sampling labels
Matthew David Pitkin approved this merge request 4 years ago

approved this merge request
Moritz Huebner approved this merge request 4 years ago

approved this merge request
Moritz Huebner merged 4 years ago

merged
Moritz Huebner mentioned in commit 819a0635 4 years ago

mentioned in commit 819a0635
Moritz Huebner changed milestone to %1.1.0 4 years ago

changed milestone to %1.1.0

Please register or sign in to reply