-
Notifications
You must be signed in to change notification settings - Fork 244
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
Update build for gfsv17 test to use -D32BIT=ON and set fewer tests for S4 #1728
Update build for gfsv17 test to use -D32BIT=ON and set fewer tests for S4 #1728
Conversation
No changes in gnu baseline, log: Comparing against new baseline for intel, log: |
Will this change allow the test to be successful on wcoss2.intel ? |
@JessicaMeixner-NOAA please bring these up to date with respective authoritative repositories
|
Unsure, I will test and let you know. |
@BrianCurtis-NOAA it is successful on WCOSS2 now. I just ran the 2 32bit tests and they matched the baseline that I created. I'm happy to share paths if needed. |
Thanks! Can you update rt.conf enabling wcoss2.intel for that test? |
@BrianCurtis-NOAA I think it's enabled by moving it to the other compile section already, but if I'm missing something here let me know, I'm happy to update this. |
@BrianCurtis-NOAA tested on wcoss2 and updated rt.conf |
@JessicaMeixner-NOAA We can start working on this pr. Can you sync up? #1701 contains a minor update on rt.conf. It will be great to combine #1701 into this pr. @DavidHuber-NOAA can you create a direct pr to Jessica's branch? |
Branch is updated @jkbk2004 |
on-behalf-of @ufs-community <jong.kim@noaa.gov>
on-behalf-of @ufs-community <ecc.platform@noaa.gov>
Jenkins-ci logs attached, ORTs passed. |
WCOSS2 in maintenance mode. Acorn had a hardware failure on their ecflow node so i can't finish RT's, but baselines are done. Please skip both. |
Ok! We can start merging this pr. Please, go ahead for final reviews and approvals. |
Description
Move cpld_control_gfsv17 to -D32BIT=ON build section as GFSv17 will use this option. Description of the evaluation by @LydiaStefanova-NOAA can be found here comparing -D32BIT=ON vs OFF for HR1 runss.
Top of commit queue on: TBD
Input data additions/changes
Anticipated changes to regression tests:
Subcomponents involved:
Combined with PR's (If Applicable):
Commit Queue Checklist:
Linked PR's and Issues:
Testing Day Checklist:
Testing Log (for CM's):