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

Update WRF scripting and tutorial for derecho #636

Merged
merged 31 commits into from
Mar 13, 2024
Merged

Conversation

braczka
Copy link
Contributor

@braczka braczka commented Feb 10, 2024

Description:

The csh scripting templates used to run WRF-DART and WRF-DART tutorial need to be updated from Cheyenne to Derecho. This PR includes functionality to get WRF functionality for version 3 (3.9.1), which was the version originally used when designing the tutorial. WRF Version 4 functionality will be included in a separate PR.

Fixes issue

This fixes issue #627.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Documentation changes needed?

  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.

Tests

I ran the existing WRF-DART Tutorial on Derecho and verified I could repeat the CONUS WRF test case and results. I compiled the WRFv3.9.1, WPSv3.9.1 and WRFDAv3.9.1 executables on Derecho using gfortran as covered in issue #627. Extending functionality beyond v3 to v4 will be included in a separate PR.

Checklist for merging

  • Updated changelog entry
  • Documentation updated
  • Update conf.py

Checklist for release

  • Merge into main
  • Create release from the main branch with appropriate tag
  • Delete feature-branch

Testing Datasets

The base directory for my WRF-Tutorial test case is located here: /glade/derecho/scratch/bmraczka/WRFv3.9.1_DART_Tutorial

  • Dataset needed for testing available upon request
  • Dataset download instructions included
  • No dataset needed

@braczka braczka added Derecho issues related to running on NCAR's new supercomputer wrf Weather Research & Forecasting Model labels Feb 10, 2024
@braczka braczka self-assigned this Feb 10, 2024
@braczka braczka marked this pull request as ready for review February 11, 2024 00:52
@braczka braczka marked this pull request as draft February 11, 2024 00:55
@braczka braczka marked this pull request as ready for review February 12, 2024 15:41
@braczka
Copy link
Contributor Author

braczka commented Feb 12, 2024

Making this ready for review. Much of the scripting changes are self explanatory. For the mpi run command for either filter or the WRF executables I used the mpiexec command following Derecho documentation which specifies the number of processors and processors per node, but unsure I need to be this specific. CISL staff has resolved the module load difficulties I was having with the csh scripts (both command line execution and PBS submission) thus I did not need to include any special instructions there.

Because the observations were pre-prepared for the tutorial I did not troubleshoot or edit the observations section of the tutorial (Step 3). This does involve use of the prep_bufr obs converter, which I know there is active work being done on this #634. This may need to be incorporated in this pull request, not sure.

Within the documentation I try to give guidance on how to compile WRF and DART, even referring to a previous github issue where I go into details on environment, source code changes and configuration options. Not sure this is needed, given WRF is not our primary responsibility, but I think its helpful for DART users given the recent transition to Derecho. This PR provides a working version for WRFv3.9.1, and am still testing compatibility for WRVv4.x. This will be a separate PR.

@mjs2369
Copy link
Contributor

mjs2369 commented Feb 27, 2024

At the locations where we evaluate the logical if ( $SUPER_PLATFORM == 'yellowstone' ) then, should we use if ( $SUPER_PLATFORM == 'LSF system' ) then or something simalar instead since Yellowstone is decommissioned?

@mjs2369
Copy link
Contributor

mjs2369 commented Feb 27, 2024

Lastly, there are several issues related to "Step 3: Prepare observations [OPTIONAL]" part of the tutorial and the corresponding documentation page for PROGRAM prepbufr

These do not need to be addressed as a part of this PR.

Many of the prepqm files are “cosblocked”, which makes it so you can't untar them. On cheyenne there used to be a cosconvert program to fix the file, but this is not on Derecho. The majority of these prepqm files are therefore currently unusable. I followed the instructions here to build the program, but it was missing necessary files. Nancy was able to get the source code over email and build successfully. She is awaiting a response to her question of if we could put the source code some place users could access it. EDIT: we will not make the cosconvert source available or point to it - this is an RDA problem, not a DART problem

After discussion, we want to combine the information about prepbufr in this section of the WRF-DART tutorial with the info in the doc for the prepbufr program. It is difficult to follow the instructions and get all the necessary info having to go back and forth between these two documents.

In a separate pull request for the prepbufr documentation, we will remove the info in this section of the tutorial and instead just have it point to the prepbufr program page so we don't have to maintain two separate docs for this. It would also be good to include a note in the WRF-DART tutorial to use prepbufr obs from the year that matches the wrfinput files that come with the tutorial (2017)

We will also need to address several issues with the PROGRAM prepbufr page in addition to combining these two bits of documentation. See issue #634 for a list of these

Cheyenne no longer exists. 
Also NCAR -> NSF ncar see #584 for details
@braczka
Copy link
Contributor Author

braczka commented Mar 8, 2024

Thanks @mjs2369 -- based on our last software standup discussion the prepbufr code discussion/resolution will continue on #634, but this PR can go forward as the WRF Tutorial documentation includes the converted obs already. I will add another another commit that includes some comments/warning within the observation section WRF documentation to keep users apprised of the prepbufr developments.

braczka added 2 commits March 8, 2024 10:36
Onging work for prepbufr obs conversion
is in bug report #634.  The WRF-Tutorial
will function without full resolution of
prepbufr obs conversion for Derecho.
@braczka
Copy link
Contributor Author

braczka commented Mar 8, 2024

@mjs2369 Take one last look and then please merge into main at your discretion next week.

@mjs2369
Copy link
Contributor

mjs2369 commented Mar 11, 2024

At the locations where we evaluate the logical if ( $SUPER_PLATFORM == 'yellowstone' ) then, should we use if ( $SUPER_PLATFORM == 'LSF system' ) then or something simalar instead since Yellowstone is decommissioned?

@braczka what do you think about this?

@braczka
Copy link
Contributor Author

braczka commented Mar 11, 2024

At the locations where we evaluate the logical if ( $SUPER_PLATFORM == 'yellowstone' ) then, should we use if ( $SUPER_PLATFORM == 'LSF system' ) then or something simalar instead since Yellowstone is decommissioned?

@braczka what do you think about this?

Yes -- this seems reasonable to me, just make sure to do a search/replace for all instances of yellowstone in the code. I think I also mention Yellowstone instead of LSF system in the documentation as well.

Copy link
Contributor

@mjs2369 mjs2369 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All comments have been addressed. Great work on this Brett.

Approved

@mjs2369 mjs2369 added the release! bundle with next release label Mar 13, 2024
@mjs2369 mjs2369 merged commit fd89302 into main Mar 13, 2024
4 checks passed
@mjs2369 mjs2369 deleted the WRF_Tutorial_Derecho branch March 13, 2024 17:41
@hkershaw-brown hkershaw-brown mentioned this pull request Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Derecho issues related to running on NCAR's new supercomputer release! bundle with next release wrf Weather Research & Forecasting Model
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants