BUGFIX: fix time+calibration marginalization
The time+calibration marginalization has been broken for a while, there are a few fixes in there for that. I also pulled out the creation of the lookup table in order to make it easier to integrate downstream with bilby_pipe.
Edited by Colm Talbot
Merge request reports
Activity
added >100 lines Bug labels
mentioned in merge request bilby_pipe!508 (merged)
changed milestone to %2.0.0
added 20 commits
-
bb7cc954...b6f385b7 - 16 commits from branch
master
- 406c4924 - Merge remote-tracking branch 'origin' into bugfix-calibration
- 7f16309d - DEV: add class for precomputed calibration curves
- 65d9d577 - TEST: improve calibration testing
- 5e899fa6 - EXAMPLE: update calibration example
Toggle commit list-
bb7cc954...b6f385b7 - 16 commits from branch
- Resolved by Colm Talbot
I'm not sure why this test is failing, but I noticed a larger problem that is resolved in !1211 (merged).
added 7 commits
-
d769a814...67d30204 - 6 commits from branch
master
- 3d008115 - Merge remote-tracking branch 'origin' into bugfix-calibration
-
d769a814...67d30204 - 6 commits from branch
added 4 commits
-
3d008115...b59a8e60 - 2 commits from branch
master
- edeab6c8 - Merge remote-tracking branch 'origin' into bugfix-calibration
- 9bd97a82 - TEST: remove time/make relbin test more precise
-
3d008115...b59a8e60 - 2 commits from branch
added 1 commit
- fb2c5072 - REVERT: remove precomputed calibration model
mentioned in merge request !1215 (merged)
added 5 commits
-
fb2c5072...ee78d544 - 4 commits from branch
master
- 734d309f - Merge branch 'master' into 'bugfix-calibration'
-
fb2c5072...ee78d544 - 4 commits from branch
added 6 commits
-
734d309f...6c73e287 - 5 commits from branch
master
- 74728c70 - Merge branch 'master' into 'bugfix-calibration'
-
734d309f...6c73e287 - 5 commits from branch
requested review from @gregory.ashton, @sylvia.biscoveanu, @michael.williams, and @soichiro.morisaki
- Resolved by Colm Talbot
Please register or sign in to reply