dtcenter / METdataio

https://metdataio.readthedocs.io/en/latest/index.html
Apache License 2.0
5 stars 3 forks source link

Enhance METdataio schema to handle new/modified MPR and ORANK columns introduced for MET 12.0.0 beta6 #320

Closed JohnHalleyGotway closed 2 weeks ago

JohnHalleyGotway commented 1 month ago

Describe the Enhancement

Issue dtcenter/MET#2924 and corresponding pull requests dtcenter/MET#2939 and dtcenter/MET#2942 made changes to the MPR and ORANK line types. The METdataio schema needs to be updated to handle the diffs described below:

  1. In MPR, CLIMO_MEAN is renamed OBS_CLIMO_MEAN.
  2. In MPR, CLIMO_STDEV is renamed OBS_CLIMO_STDEV.
  3. In MPR, CLIMO_CDF is renamed OBS_CLIMO_CDF.
  4. In MPR, new FCST_CLIMO_MEAN and FCST_CLIMO_STDEV are added to the end of the line.
  5. In ORANK, CLIMO_MEAN is renamed OBS_CLIMO_MEAN.
  6. In ORANK CLIMO_STDEV is renamed OBS_CLIMO_STDEV.
  7. In ORANK, new FCST_CLIMO_MEAN and FCST_CLIMO_STDEV are added to the end of the line.

Please see updated MPR line type and ORANK line type tables.

@bikegeek, it is not clear to me if changes are needed in METplotpy, METcalcpy, and METviewer. I'm not sure if/how data from these columns are used in those components. Please let me know if you'd like me to write up additional issues for them.

METreformat code for the MPR linetype is impacted due to the addition of new columns: https://github.com/dtcenter/METdataio/issues/321

Time Estimate

1 day?

Sub-Issues

Relevant Deadlines

List relevant project deadlines here or state NONE.

Funding Source

Define the source of funding and account keys here or state NONE.

Define the Metadata

Assignee

Labels

Projects and Milestone

Define Related Issue(s)

Consider the impact to the other METplus components.

Enhancement Checklist

See the METplus Workflow for details.

Select: Repository level development cycle Project for the next official release Select: Milestone as the next official version

bikegeek commented 1 month ago

Created issue https://github.com/dtcenter/METdataio/issues/321 for the METreformat module to address additional columns added to the MPR linetype

JohnHalleyGotway commented 4 weeks ago

@bikegeek FYI, the dtcenter/MET#2924 issue was completed today and the final PR merged. You can find updated MPR and ORANK data tomorrow in the output from the MET nightly build run on seneca.