Feature/regional/condense mesoscale#970
Conversation
| print(f"ERROR: Undefined data type for missing data file: {info[1]}" | ||
| + f"\nPlease edit the get_data_type() function in" | ||
| + f" USHevs/mesoscale/mesoscale_util.py") | ||
| + f" USHevs/cam/mesoscale_util.py") |
There was a problem hiding this comment.
If we are removing all instances of "mesoscale", should this script be renamed "cam_util.py"? However, I realize there might already exist a script by that name so in that case, it makes sense to keep it "mesoscale_util.py" to avoid overwriting.
There was a problem hiding this comment.
@ShannonShields-NOAA That's a good question. I kept "mesoscale" in this instance for simplicity and to avoid accidentally mixing rap and cam utilities. mesoscale_util.py is used in a lot of RAP verification scripts. I could maybe change it to "rap_util.py"?
There was a problem hiding this comment.
@MarcelCaron-NOAA I'll let @AndrewBenjamin-NOAA chime in about how we should address this. I think it depends on if we need to remove all instances of mesoscale to avoid any confusion with NCO...
|
@MarcelCaron-NOAA For testing purposes, what VDATE should I use? Is the default ok or should I set a specific date? |
The defaults should be fine! |
|
@MarcelCaron-NOAA I have run the following stats jobs. Everything looks good to me with no ERRORs or WARNINGs and I see output stat files. Please confirm. jevs_stats_cam_rap_grid2obs_00Log: /lfs/h2/emc/vpppg/noscrub/shannon.shields/pr970test/EVS/dev/drivers/scripts/stats/cam/jevs_stats_cam_rap_grid2obs_00.o120234053 jevs_stats_cam_rap_precip_00Log: /lfs/h2/emc/vpppg/noscrub/shannon.shields/pr970test/EVS/dev/drivers/scripts/stats/cam/jevs_stats_cam_rap_precip_00.o120234250 jevs_stats_cam_rap_snowfall_00Log: /lfs/h2/emc/vpppg/noscrub/shannon.shields/pr970test/EVS/dev/drivers/scripts/stats/cam/jevs_stats_cam_rap_snowfall_00.o120234493 |
|
@ShannonShields-NOAA Agreed, logs are clean and stats look normal compared to the parallel (some minor differences related to develop/rm_nam_hires_sref branch differences and not related to this PR). 👍 stats test jobs look good, no concerns from me |
|
Excellent! Since my workday is ending soon, I will test the plot jobs tomorrow. |
|
Sounds good! |
|
@MarcelCaron-NOAA Some of the plot jobs are done running; please check and confirm everything looks right. I saw the expected WARNINGs about the rrfsmem data: I can also confirm I saw the expected message for time-shifted RAP verification: jevs_plots_cam_headlineLog: /lfs/h2/emc/vpppg/noscrub/shannon.shields/pr970test/EVS/dev/drivers/scripts/plots/cam/jevs_plots_cam_headline.o120382659 jevs_plots_cam_snowfall_last90daysLog: /lfs/h2/emc/vpppg/noscrub/shannon.shields/pr970test/EVS/dev/drivers/scripts/plots/cam/jevs_plots_cam_snowfall_last90days.o120382307 |
|
@MarcelCaron-NOAA The precip plot job has finished (the last plot job of grid2obs will take the rest of the day it looks like). jevs_plots_cam_precip_last90daysLog: /lfs/h2/emc/vpppg/noscrub/shannon.shields/pr970test/EVS/dev/drivers/scripts/plots/cam/jevs_plots_cam_precip_last90days.o120382163 |
|
@ShannonShields-NOAA Thanks for running these. I can see I missed a few things
|
|
@MarcelCaron-NOAA Yes, I will re-run the snowfall plot job. Should I re-run the precip job or stop and re-run the grid2obs job? |
|
The precip plots job completed cleanly and output graphics look normal to me. Here's an example graphic with the RAP line included: The graphic at this link shows what the RRFS Member lines will look like for comparison. ✅ jevs_plots_cam_precip_last90days |
I think rerunning snowfall and maybe headline as well is sufficient to confirm the two changes! So no need in my opinion |
Also, just a note that grid2obs plots took about 3 hours to finish in my tests, a lot less than its walltime. Not plotting the five RRFS Members reduces the job's runtime substantially. |
|
@MarcelCaron-NOAA Here is the output for the re-runs of headline and snowfall plots. jevs_plots_cam_headlineLog: /lfs/h2/emc/vpppg/noscrub/shannon.shields/pr970test/EVS/dev/drivers/scripts/plots/cam/jevs_plots_cam_headline.o120392147 jevs_plots_cam_snowfall_last90daysLog: /lfs/h2/emc/vpppg/noscrub/shannon.shields/pr970test/EVS/dev/drivers/scripts/plots/cam/jevs_plots_cam_snowfall_last90days.o120392164 |
|
Thanks @ShannonShields-NOAA! Both jobs completed cleanly and their output look normal. Sample graphics look normal and show RAP statistics as expected. Also as expected, no DEBUG statements in the log about the 'shift' query. grid2obs is still running, but I expect it to complete by noon today. |
|
@MarcelCaron-NOAA The grid2obs plot job is still running, but I thought I'd check the log file so far and noticed the following KeyError: It seems something might be wrong with the calculate_stat function for performance diagrams? jevs_plots_cam_grid2obs_last90daysLog: /lfs/h2/emc/vpppg/noscrub/shannon.shields/pr970test/EVS/dev/drivers/scripts/plots/cam/jevs_plots_cam_grid2obs_last90days.o120381934 |
|
@ShannonShields-NOAA Thanks for catching this early. It looks like this is happening for the PTYPE graphic, and is related to an issue plotting MCTC statistics. I'll do some more testing and will push a fix before the end of the workday. |
|
@ShannonShields-NOAA I made a fix and confirmed in my tests that the fix clears up the remaining errors and any warnings about sample equalization. Ready to continue testing after the weekend. Thanks! |

Note to developers: You must use this PR template!
Description of Changes
This PR removes the mesoscale component from EVS and moves RAP verification into cam. Some details:
Developer Questions and Checklist
Yes; code freeze date for RRFS work is currently set for June 17.
No
No
${USER}where necessary throughout the code.HOMEevsare removed from the code.dev/drivers/scriptsordev/modulefileshave been made in the correspondingecf/scriptsandecf/defs/evs-nco.def?Testing Instructions
1) Clone this branch for testing
cdinto any testing location on WCOSS2-dev, e.g./lfs/h2/emc/vpppg/noscrub/${USER}2) Which jobs to test
${driver}" hereafter):[Total: 3 stats jobs; 4 plots jobs]
3) Set up jobs
... where
${STEP}isprep,stats, orplots, and${driver}is the name of the jobFor all drivers:
HOMEevs - set to your test EVS directory
COMOUT - set to your test output directory
KEEPDATA - (optional) set to
"YES"SENDMAIL - (optional) set to
"NO"DATAROOT - (optional) set to your test DATAROOT directory
For stats drivers:
COMIN - set to
/lfs/h2/emc/vpppg/noscrub/emc.vpppg/$NET/$evs_ver_2dFor plots drivers:
COMIN - set to
/lfs/h2/emc/vpppg/noscrub/marcel.caron/test/condense_mesoscale4) Test jobs
cdinto your test directory (where you want the log files to go)Submit jevs_stats_cam_rap_grid2obs using
qsub -v vhr=08 ${driver}.shSubmit all other drivers using
qsub -v vhr=00 ${driver}.shWe can discuss whether or not we want to test anything else.
5) Check jobs
6) During testing ...