Skip to content
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

load balance wave tests #461

Closed
JessicaMeixner-NOAA opened this issue Mar 10, 2021 · 8 comments · Fixed by #544
Closed

load balance wave tests #461

JessicaMeixner-NOAA opened this issue Mar 10, 2021 · 8 comments · Fixed by #544
Assignees
Labels
enhancement New feature or request

Comments

@JessicaMeixner-NOAA
Copy link
Collaborator

Description

Update load balancing for wave related tests.

Solution

It has been requested that tests with waves match the timing of the tests without waves.

@JessicaMeixner-NOAA JessicaMeixner-NOAA added the enhancement New feature or request label Mar 10, 2021
@JessicaMeixner-NOAA JessicaMeixner-NOAA self-assigned this Mar 10, 2021
@junwang-noaa
Copy link
Collaborator

The two wave tests fv3_gfdlmprad and fv3_gfdlmprad_atmwav did fail in recent code commit as they took too long to run. Currently the low resolution RT tests (1 degree fv3 and fv3+wave) have run time limit 20mins.

  • The standalone fv3 tests without wave using 150tasks take 50s to run,
  • fv3(150tasks)+ww3(42tasks) test takes 650s to run, two way coupling test takes 800s to run. These tests sometimes failed with 900s wall clock time limit in regression test.
  • Increasing ww3 to use 82 tasks one way coupling test will take about 350s.
  • Increasing ww3 to use 162 tasks one way coupling test will take about 200s.

@JessicaMeixner-NOAA @aliabdolali The ww3 still causes delays in the fv3-ww3 low resolution runs, is this what you expect? Do you have any suggestion what can be done? It seems to me low resolution ww3 scales not as good as to the higher resolutions ww3 run in coupled C384 runs, is there any reason? Also I just increased the ww3 tasks without changing any other ww3 settings, please let me know if additional changes are required.

To resolve the low resolution wave test issue in UFS RT, I think we will increase the ww3 tasks from 42 to 82 at this time. Please let us know if you have any suggestions. Thanks

@aliabdolali
Copy link
Collaborator

@junwang-noaa Do you want to make the grid coarser? I have a 2-degree global grid, so we can replace it in the setup to reduce computations.
Also, I am working to add standalone regression tests similar to UFS setups to WW3. Once it is ready, I can tell you the computational time in stand-alone WW3. Note that these tests are supposed to check restart b4b.

@junwang-noaa
Copy link
Collaborator

@aliabdolali FV3 is running 1 degree, so I think 1 degree ww3 should be fine. If you have standalone ww3 RT set up, please let us know the computational time. So far the one way coupling ww3 results are identical when using the different ww3 mpi tasks.

@JessicaMeixner-NOAA
Copy link
Collaborator Author

We need to add more nodes to the wave model. My suggestion would be to use the ESMF profiling tools and make sure the increase to 82 from 48 is sufficient.

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Apr 7, 2021 via email

@JessicaMeixner-NOAA
Copy link
Collaborator Author

If there's a maximum number of nodes that WW3 is allowed, and you would like it to run in a certain time period, we likely will need to explore even lower resolution for WW3 as @aliabdolali suggested. If you give us a maximum number of nodes we can design a wave grid to fit that or we we will need to use more nodes for the waves.

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Apr 7, 2021 via email

@aliabdolali
Copy link
Collaborator

@junwang-noaa @JessicaMeixner-NOAA
Let me check the code speed for ww3 standalone on this setup, from there, we can decide.
Also, I suggested yesterday that a 2-degree global grid is available. So we can use it for faster computation.
AA

pjpegion pushed a commit to NOAA-PSL/ufs-weather-model that referenced this issue Apr 4, 2023
…unity#461)

* modify FV3GFS_io.F90 by fixing errors associated with initializing snow depth over ice in the case where both land and water coexist (i.e. fractional grid case)
* z0ice is changed to 1.0 cm from 1.1cm in atmos_model.F90
epic-cicd-jenkins pushed a commit that referenced this issue Apr 17, 2023
* Modify path to external GFS data
* Modify path to external GFS data for WE2E
epic-cicd-jenkins pushed a commit that referenced this issue Apr 17, 2023
* [develop] Added return code to srw_build.sh script.

* Correct issue in sorc/CMakeLists.txt so that the ufs-model will build again.

* Restore PID functionality in tests/build.sh for GitHub Actions autoci builds.

* Add return code to tests/build.sh and read the code in srw_build.sh.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
3 participants