Edits to merge GHI NRT and NRT-Streamflow - #1769
Draft
emkemp wants to merge 21 commits into
Draft
Conversation
With these changes, "NRT" includes both "NRT" and "NRT-Streamflow" variables written by LIS in 557WW streamflow convention. The SURFACEMODEL data (in 1d tilespace) is then postprocessed by LVT and related scripts. With these changes, there are two support NRT configurations: NOAH-RAPID and NOAHMP-RAPID. Additional configurations can be added later as needed.
This allows the user to specify separate output methodologies for SURFACEMODEL and ROUTING output files. If this new option is not specified, the routing option will fall back on the existing generic "Output methodology:" option in lis.config. Updated MODEL_OUTPUT_LIS.TBL files for noah39 and noahmp401 to add HYMAP output variables.
NOTE: make_lvt_config_*hr_noah*.py scripts are set to use RAPID. In the future, this could be made more flexible.
This is done in 557post runmode, and only for GRIB2 output.
TimeAvgOpt will write both a time averaged variable (like Qs_tavg) and an accumulated value (like Qs_acc). This is required for merging the NRT and NRT-STR workflows since both variants would be required from a single run.
Code compiles, have not tested yet.
Name change emphasizes that this supports the postprocessed (unified) NRT GRIB2 files produced by LVT. This version removes support for JULES, and assumes LIS is running coupled with RAPID. Also, the file name convention is revised to meet latest request from LCMC and Shyft -- namely, the filename indicates the start date, cycle hour, and "forecast hour" of the LIS output. This makes it much easier to discriminate between LIS runs. The _GP. block is revised to indicate "LIS-NRT-NOAH-RAPID" or "LIS-NRT-NOAHMP-RAPID". The GHI MR runs will replace "NRT" in the filename, again to make it easier to discriminate output files. NOAHMP variable list is extended to include PotEvap_tavg. Support for LIS HYMAP can be added in the future. So far, only tested with 3-hrly LIS-NRT-NOAH-RAPID files.
…ion. Logic now searches for 03 and 06 hr LIS "forecasts" from earlier LIS cycles.
Still TODO: Handle unit conversion for qs_tavg and qsb_tavg.
For 557 WW, LIS will output both qs{b}_tavg and qs{b}_acc, but LIS will
require the same units. Our solution is to output in kg m-2, and have
LVT convert to kg/m2s. This only happens if:
* Runmode is 557 post
* lvt.config specifies qs_tavg or qsb_tavg in kg/m2s
* The netCDF file has units of kg m-2.
This mostly unifies the NRT and NRT-STR workflows. TODO: Modify run_ncks.py to merge "NRT-STR" style netCDF files. TODO: Changes for NoahMP401.
Only handles SURFACEMODEL files.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Changes to allow merger of GHI NRT and NRT-Streamflow workflows. Includes changes to file name conventions.