Voyager parameter update
Merge request reports
Activity
added 4 commits
-
275c59e5...9ff4ba54 - 2 commits from branch
master
- c5083be5 - Revise suspension model
- 50cddebb - Voyager parameter update
-
275c59e5...9ff4ba54 - 2 commits from branch
added 26 commits
-
50cddebb...b69d4267 - 25 commits from branch
master
- 27a26773 - Voyager parameter update
-
50cddebb...b69d4267 - 25 commits from branch
@jameson.rollins This should be ready to merge.
Shouldn't it have failed the CI since it changes the Voyager noise?
@christopher.wipf this MR was submitted before any global approval rules were in place, so no approval was required for this MR, which meant the 'noise_change_approval' job succeeded. I added an approval rule and re-ran the job and it's now failing because of no approval, as it should. new MR from now shouldn't have this problem since now all MR require at least one approval.
OK, that makes sense.
Please review and approve when you get a chance. (Note this was previously reviewed as part of !61 (closed) before being split off.) It'll be quite helpful for those of us interested in Voyager to have a reasonably up to date parameter set available in gwinc.
added 9 commits
-
27a26773...15bac0fb - 8 commits from branch
master
- 268c3318 - Voyager parameter update
-
27a26773...15bac0fb - 8 commits from branch
added 2 commits
mentioned in commit 9f52b9e9