-
Notifications
You must be signed in to change notification settings - Fork 698
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
{geo}[foss/2023a] WRF v4.5.1 w/ dmpar #18741
{geo}[foss/2023a] WRF v4.5.1 w/ dmpar #18741
Conversation
…5.1_netCDF-Fortran_separate_path.patch
@boegelbot please test @ generoso EB_ARGS= --include-easyblocks-from-pr 2997 |
@ocaisa: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1711290993 processed Message to humans: this is just bookkeeping information for me, |
@boegelbot please test @ jsc-zen2 EB_ARGS="--include-easyblocks-from-pr 2997" |
@ocaisa: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1711295242 processed Message to humans: this is just bookkeeping information for me, |
This is still a draft. That patch is still not correct, and one test is failing. |
@stefan-wolfsheimer Sure, I just wanted to check the easyblock change as it is pretty trivial, but I still need the test case. I'll come back to it once you change the status |
@stefan-wolfsheimer: I noticed your comment, but I only dance when @akesandgren or @bartoldeman or @bedroge or @boegel or @branfosj or @casparvl or @jfgrimm or @lexming or @Micket or @migueldiascosta or @ocaisa or @SebastianAchilles or @smoors or @verdurin or @robert-mijakovic or @deniskristak or @ItIsI-Orient or @PetrKralCZ or @sassy-crick or @laraPPr tells me (for now), I'm sorry... - notification for comment with ID 1711375449 processed Message to humans: this is just bookkeeping information for me, |
@boegelbot please test @ generoso EB_ARGS="--include-easyblocks-from-pr 2997" |
@ocaisa: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1721225308 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@stefan-wolfsheimer The generoso build failed on the sha256 for the sources, |
@boegelbot please test @ jsc-zen2 EB_ARGS="--include-easyblocks-from-pr 2997" |
@ocaisa: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1721476775 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
1c6024e
to
ad2faf9
Compare
@boegelbot please test @ generoso |
Test report by @boegel |
@boegel: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1874318410 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @boegelbot |
@boegelbot please test @ jsc-zen3 |
@bedroge: 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 2165339670 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ generoso |
@bedroge: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2165606773 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ generoso |
@bedroge: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2165619243 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @bedroge |
Test report by @bedroge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Going in, thanks @stefan-wolfsheimer! |
(created using
eb --new-pr
)requires:
WRF*
subdirectory for WRF versions >= 4.5.1 easybuild-easyblocks#2997depends on: