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 case-gen scripts for LAM capability #499

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

hertneky
Copy link
Collaborator

Running the case generation on LAM output from UFS SRW does not currently work. These updates fix that.

Sample LAM data can be found on Hera: /scratch1/BMC/gmtb/Tracy.Hertneky/phys_tne/socrates/2018020412

Sample case over the S. Ocean was created with the following command:

./UFS_forcing_ensemble_generator.py -d /scratch1/BMC/gmtb/Tracy.Hertneky/phys_tne/socrates/2018020412 -n socrates -lons 133 -lats -51.5 -dt 75 -cres 768 -sdf SCM_GFS_v16 -sc -lam -fm 2 -vm 2

./run_scm.py --npz_type gfs --file scm_ufsens_socrates.py --timestep 75 -l 64 --n_itt_out 48 --n_itt_diag 48

Comparison of SCM to UFS column:
Diffs could be attributable to diff suite used (HRRR_gf for UFS vs GFS_v16 for SCM, since HRRR_gf was not working from case_gen input)
Screenshot 2024-07-31 at 3 04 52 PM

Copy link
Collaborator

@grantfirl grantfirl left a comment

Choose a reason for hiding this comment

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

Looks good to me as long as you've tested on LAM output.

@hertneky
Copy link
Collaborator Author

hertneky commented Aug 1, 2024

Tested using LAM output from Anders' SOCRATES case. Only tested fm=2, vm=2 (default). I can test other options if desired (just not "exact" mode).

Copy link
Collaborator

@dustinswales dustinswales left a comment

Choose a reason for hiding this comment

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

These changes look identical to what I had done.

@grantfirl grantfirl merged commit 1e3475c into NCAR:main Aug 19, 2024
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants