-
Notifications
You must be signed in to change notification settings - Fork 349
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Avoid pegasus 5.0.7 #4651
Avoid pegasus 5.0.7 #4651
Conversation
@spxiwh I think we need to have the pegasus binaries also be version constrained e.g. the ones installed for the workflow tests. I don't recall how the lvk images are done, so might be worth checking if that is already pinned. |
@spxiwh Unless of course, the bug doesn't appear if you just use the old api version, but the newer planner version. I didn't test that myself. Does that work? |
This is something in the planner, so yeah, pinning the API isn't doing anything .... |
This reverts commit c9c915f.
@ahnitz This fixes the CI ... It should be reverted once the pegasus 5.0.8 release is made. I've asked LVK clusters not to upgrade to the 5.0.7 executables. Others using PyCBC+pegasus should also stick at 5.0.6 until a new release is available. |
* Avoid pegasus 5.0.7 * Needs to be here, not in API requirements * Revert "Avoid pegasus 5.0.7" This reverts commit c9c915f.
* Avoid pegasus 5.0.7 * Needs to be here, not in API requirements * Revert "Avoid pegasus 5.0.7" This reverts commit c9c915f.
* Avoid pegasus 5.0.7 * Needs to be here, not in API requirements * Revert "Avoid pegasus 5.0.7" This reverts commit c9c915f.
* Avoid pegasus 5.0.7 * Needs to be here, not in API requirements * Revert "Avoid pegasus 5.0.7" This reverts commit c9c915f.
* Use more memory efficient reading of trigger files * Optimize pycbc_page_coincinfo * Optimize pycbc_page_coinc_snrchi * Optimize pycbc_page_snrchi * Optimize pycbc_plot_hist * Merge in 4662 * Avoid pegasus 5.0.7 (#4651) * Avoid pegasus 5.0.7 * Needs to be here, not in API requirements * Revert "Avoid pegasus 5.0.7" This reverts commit c9c915f. * use macos 12 (#4722) * use macos 12 * update --------- Co-authored-by: Alex Nitz <alex.nitz@gmail.com>
This version has a bug we don't want. The 5.0.8 release should be fine. Did I miss any file?