search

NOAA-EMC / RDASApp

Regional DAS
GNU Lesser General Public License v2.1
0 stars 5 forks source link

Update RDASApp to use git submodules + Rocky 8 for Hera #11

Closed CoryMartin-NOAA closed 1 month ago

CoryMartin-NOAA commented 2 months ago

This PR is intended to replace #9 and only includes git submodules and build changes.

I tested and it compiles for the FV3 case on Hera on the Rocky8 login nodes. I did not yet test for MPAS or on Orion.

TingLei-NOAA commented 2 months ago

@CoryMartin-NOAA Thanks for providing a version working with Rocky 8. Does this mean this branch will work after tomorrow's hera upgrade to Rocky8?

CoryMartin-NOAA commented 2 months ago

@TingLei-NOAA it works now if you log in to one of the rocky8 nodes, after tomorrow's maintenance, those will be the default option.

TingLei-NOAA commented 1 month ago

The clone and building of rdas of this PR on hera had succeeded. Though mpi-jedi related components are missing, it is fine. We can add it later as done in https://github.com/NOAA-EMC/RDASApp/pull/9. Also, as our discussion on rdas including those around https://github.com/NOAA-EMC/RDASApp/pull/9, I think it is great to have a RDAS repo to work as a starting point. Further discussion/collaboration could continue by more and more use/developments of RDASapp on github. The PR is approved and Thanks to @CoryMartin-NOAA for this effort.

CoryMartin-NOAA commented 1 month ago

@TingLei-NOAA did you try it for mpas, did it work?

TingLei-NOAA commented 1 month ago

@CoryMartin-NOAA running "build.sh -f -a -d -t hera" , mpas (mpas-jedi) is not built in bundle dir. But I saw mpas and mpas-jedi are in sorc dir as the submodules. Maybe some options should be turned on? I will have a look at that later.

CoryMartin-NOAA commented 1 month ago

./build.sh -m mpas should do the trick (I hope!)

ShunLiu-NOAA commented 1 month ago

@CoryMartin-NOAA Thank you for updating RDASApp!