-
Notifications
You must be signed in to change notification settings - Fork 706
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
{astro,lib}[foss/2023b] EveryBeam v0.6.1, WSClean v3.5, DP3 v6.2 #21765
base: develop
Are you sure you want to change the base?
{astro,lib}[foss/2023b] EveryBeam v0.6.1, WSClean v3.5, DP3 v6.2 #21765
Conversation
Updated software
|
@boegelbot please test @ generoso |
@casparvl: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2468410804 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ generoso |
@casparvl: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2468471070 processed Message to humans: this is just bookkeeping information for me, |
@boegelbot please test @ jsc-zen3 |
@casparvl: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2468489307 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @boegelbot |
Test report by @casparvl |
@boegelbot please test @ generoso |
@casparvl: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2468572755 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ jsc-zen3 |
@casparvl: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2485335323 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ generoso |
@casparvl: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2488726530 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
(created using
eb --new-pr
)Created this PR because Rapthor needs a newer
WSClean
, which then in turn needs a newerEveryBeam
. Currently testing a rapthor pipeline to confirm that this combination works - then I'll mark this ready for review.Edit: confirmed to work with Rapthor, so I'd like to get this reviewed and merged :) Note that this requires breaking the one-dep-per-toolchain policy. I'm reasonably sure that these tools are typically used in combination in such pipelines though. Plus: it's the only way to make that particular pipeline work in Rapthor - I believe we find that good enough reason. I'll add exceptions for these CI checks.