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

Enhance TC-Gen to parse GTWO shapefile lead times from the data rather than assuming them to be 2, 5, and 7 days #2552

Closed
8 of 20 tasks
JohnHalleyGotway opened this issue May 24, 2023 · 0 comments · Fixed by #2553
Assignees
Labels
MET: Tropical Cyclone Tools priority: high High Priority requestor: NOAA/other NOAA Laboratory, not otherwise specified type: enhancement Improve something that it is currently doing
Milestone

Comments

@JohnHalleyGotway
Copy link
Collaborator

JohnHalleyGotway commented May 24, 2023

Describe the Enhancement

As reported by @halperin-erau, NOAA NHC has changed how it issues TC genesis probabilities in the NHC Graphical Tropical Weather Outlook (GTWO) area shapefiles, and TC-Gen should be modified to accommodate those changes. As of MET version 11.0.0, the TC-Gen tool assumes that the GTWO files always define 2, 5, and 7 day genesis probabilities, in that order. However, as of 15 May 2023, NHC now issues 2-day and 7-day genesis probabilities. The 5-day genesis probability has been discontinued.

The hard-coded setting can be seen on this line of code in the tc_gen.h header file. These lead time values listed in hours correspond to 2, 5, and 7 days.

This task is to update the logic of tc_gen.cc somewhere around this line of code to also parse the number of days from the database file (.dbf) rather than assuming 2, 5, and 7 day probs.

The gis_dump_dbf tool shows that the day values are included in the sub-record field_name entries:

> gis_dump_dbf MET_test_input/tc_data/genesis/shape/atl/202108070507/gtwo_areas_202108062331.dbf
...
|    SubRecord 2
|    |    field_name         = "PROB2DAY"
...
|    SubRecord 4
|    |    field_name         = "PROB5DAY"
...
Record 0 ...
|    BASIN    = "Atlantic"
|    AREA     = "1"
|    PROB2DAY = "30%"
|    RISK2DAY = "Low"
|    PROB5DAY = "50%"
|    RISK5DAY = "Medium"

Fixing this will require parsing the integer number of days from the field_name rather than just assuming the first is 2 days, second is 5 days, and third is 7 days.

Time Estimate

1 day

Sub-Issues

Consider breaking the enhancement down into sub-issues.
None needed.

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

  • Select engineer(s) or no engineer required
  • Select scientist(s) or no scientist required

Labels

  • Select component(s)
  • Select priority
  • Select requestor(s)

Projects and Milestone

  • Select Repository and/or Organization level Project(s) or add alert: NEED CYCLE ASSIGNMENT label
  • Select Milestone as the next official version or Future Versions

Define Related Issue(s)

Consider the impact to the other METplus components.

Enhancement Checklist

See the METplus Workflow for details.

  • Complete the issue definition above, including the Time Estimate and Funding Source.
  • Fork this repository or create a branch of develop.
    Branch name: feature_<Issue Number>_<Description>
  • Complete the development and test your changes.
  • Add/update log messages for easier debugging.
  • Add/update unit tests.
  • Add/update documentation.
  • Push local changes to GitHub.
  • Submit a pull request to merge into develop.
    Pull request: feature <Issue Number> <Description>
  • Define the pull request metadata, as permissions allow.
    Select: Reviewer(s) and Development issues
    Select: Repository level development cycle Project for the next official release
    Select: Milestone as the next official version
  • Iterate until the reviewer(s) accept and merge your changes.
  • Delete your fork or branch.
  • Close this issue.
@JohnHalleyGotway JohnHalleyGotway added type: enhancement Improve something that it is currently doing requestor: NOAA/other NOAA Laboratory, not otherwise specified alert: NEED ACCOUNT KEY Need to assign an account key to this issue MET: Tropical Cyclone Tools priority: high High Priority labels May 24, 2023
@JohnHalleyGotway JohnHalleyGotway added this to the MET 11.1.0 milestone May 24, 2023
@JohnHalleyGotway JohnHalleyGotway moved this from 📋 Backlog to 🔖 Ready in MET-11.1.0 Development May 24, 2023
JohnHalleyGotway added a commit that referenced this issue May 24, 2023
…om the dbf shapefile field names rather than assuming 2, 5, and 7 day probabilities in order.
JohnHalleyGotway pushed a commit that referenced this issue May 25, 2023
@JohnHalleyGotway JohnHalleyGotway linked a pull request May 25, 2023 that will close this issue
15 tasks
@JohnHalleyGotway JohnHalleyGotway moved this from 🔖 Ready to 👀 In review in MET-11.1.0 Development May 25, 2023
JohnHalleyGotway added a commit that referenced this issue Jun 9, 2023
Co-authored-by: John and Cindy <[email protected]>
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in MET-11.1.0 Development Jun 9, 2023
@JohnHalleyGotway JohnHalleyGotway removed the alert: NEED ACCOUNT KEY Need to assign an account key to this issue label Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MET: Tropical Cyclone Tools priority: high High Priority requestor: NOAA/other NOAA Laboratory, not otherwise specified type: enhancement Improve something that it is currently doing
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants