-
Notifications
You must be signed in to change notification settings - Fork 55
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
Flip rrtmgp to default to kokkos #2987
Conversation
…_to_default_kokkos * origin/master: (75 commits) Temporarily point to a COSP submodule branch with a bug fix Fix frontier builds Bump actions/setup-python from 5.1.1 to 5.2.0 EAMxx: Fix PRIx64 usage to include width 16. Removes preprocess function from hpp and a calls from cpp interface files Replaces preprocess interface code Fixes path for files and change c-style strings to std::strings EAMxx: use portable format for unsigned std::int64_t in printf Adds units for mass and number ofr constituent fluxes Update shell_commands to remove spa Fixes namelist to enable fine grid simulations clarify comment for cmake option add more testmods and better comments fix typo in test mod add tests for small kernels options to build sk p3 and shoc separately Edits and adds some comments to provide clarity and TODOs Removes debug print statements Adds ne30pg2 emissions files and mapping files for fine resolutions Fixes multi process test a typo in aci interface ...
Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Build InformationTest Name: SCREAM_PullRequest_Autotester_Mappy
Jenkins Parameters
Build InformationTest Name: SCREAM_PullRequest_Autotester_Weaver
Jenkins Parameters
Using Repos:
Pull Request Author: jgfouca |
Status Flag 'Pull Request AutoTester' - Jenkins Testing: 1 or more Jobs FAILED Note: Testing will normally be attempted again in approx. 2 Hrs. If a change to the PR source branch occurs, the testing will be attempted again on next available autotester run. Pull Request Auto Testing has FAILED (click to expand)Build InformationTest Name: SCREAM_PullRequest_Autotester_Mappy
Jenkins Parameters
Build InformationTest Name: SCREAM_PullRequest_Autotester_Weaver
Jenkins Parameters
SCREAM_PullRequest_Autotester_Mappy # 5818 PASSED (click to see last 100 lines of console output)
SCREAM_PullRequest_Autotester_Weaver # 6042 FAILED (click to see last 100 lines of console output)
|
@jgfouca , did you want to add labels to this PR so it can automerge once testing is complete? |
@AaronDonahue , I need to have @brhillman help me verify that the weaver DIFF is OK. |
@jgfouca - Ben is on vacation this week so don't hold your breath. |
Sounds good, we can wait on this PR until @brhillman gets back from vacation. No worries. |
@brhillman and I think that the weaver diff for |
I had to reinstate some of the copies that were removed earlier because RRTMGP performs so much better with LayoutLeft. You can force LayoutRight, but it's off by default.