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

CyclonePlotter no longer connects points to delineate storm tracks #803

Closed
20 tasks done
bikegeek opened this issue Feb 11, 2021 · 0 comments
Closed
20 tasks done

CyclonePlotter no longer connects points to delineate storm tracks #803

bikegeek opened this issue Feb 11, 2021 · 0 comments
Assignees
Labels
priority: high High Priority requestor: METplus Team METplus Development Team requestor: NOAA/other NOAA Laboratory, not otherwise specified type: bug Fix something that is not working
Milestone

Comments

@bikegeek
Copy link
Contributor

bikegeek commented Feb 11, 2021

Replace italics below with details for this issue.

Describe the Problem

Converting from Matplotlib Basemap to Cartopy has resulted in the loss of the line plot where individual storms are no longer connected to indicate storm tracks.

Expected Behavior

Provide a clear and concise description of what you expected to happen here.

Environment

Describe your runtime environment:
1. Machine: (e.g. HPC name, Linux Workstation, Mac Laptop)
2. OS: (e.g. RedHat Linux, MacOS)
3. Software version number(s)

To Reproduce

Describe the steps to reproduce the behavior:
*1. Clone the METplus code from the develop branch *
2. Follow the use case for CyclonePlotterWrapper: https://dtcenter.github.io/METplus/latest/generated/met_tool_wrapper/CyclonePlotter/CyclonePlotter.html#sphx-glr-generated-met-tool-wrapper-cycloneplotter-cycloneplotter-py
3. Use tc-pairs data on kiowa: /d1/projects/METplus/METplus_Data/development/feature_803_cycloneplotter_line
4. Generate plot and observe that only markers are getting plotted, the connecting lines between markers that are part of a storm track are missing. See below.
Screen Shot 2021-02-12 at 9 35 28 AM

Relevant Deadlines

List relevant project deadlines here or state NONE.

Funding Source

2788881

Define the Metadata

Assignee

  • Select engineer(s) Minna Win
  • Select no scientist required

Labels

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

Projects and Milestone

  • Review projects and select relevant Repository and Organization ones or add "alert:NEED PROJECT ASSIGNMENT" label
  • Select milestone to relevant bugfix version

Define Related Issue(s)

Consider the impact to the other METplus components.

Bugfix 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 main_<Version>.
    Branch name: bugfix_<Issue Number>_main_<Version>_<Description>
  • Fix the bug 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 main_<Version>.
    Pull request: bugfix <Issue Number> main_<Version> <Description>
  • Define the pull request metadata, as permissions allow.
    Select: Reviewer(s), Project(s), Milestone, and Linked issues
  • Iterate until the reviewer(s) accept and merge your changes.
  • Delete your fork or branch.
  • Complete the steps above to fix the bug on the develop branch.
    Branch name: bugfix_<Issue Number>_develop_<Description>
    Pull request: bugfix <Issue Number> develop <Description>
  • Close this issue.
@bikegeek bikegeek added type: bug Fix something that is not working alert: NEED ACCOUNT KEY Need to assign an account key to this issue requestor: METplus Team METplus Development Team labels Feb 11, 2021
@bikegeek bikegeek self-assigned this Feb 11, 2021
@bikegeek bikegeek added priority: high High Priority and removed alert: NEED ACCOUNT KEY Need to assign an account key to this issue labels Feb 11, 2021
@bikegeek bikegeek added this to the METplus-4.0 milestone Feb 11, 2021
@bikegeek bikegeek added this to To do in METplus-4.0.0-beta4 (3/2/21) via automation Feb 11, 2021
@bikegeek bikegeek moved this from To do to In progress in METplus-4.0.0-beta4 (3/2/21) Feb 12, 2021
@bikegeek bikegeek moved this from In progress to Pull Request Review in METplus-4.0.0-beta4 (3/2/21) Feb 12, 2021
@bikegeek bikegeek linked a pull request Feb 12, 2021 that will close this issue
10 tasks
METplus-4.0.0-beta4 (3/2/21) automation moved this from Pull Request Review to Done Feb 16, 2021
@TaraJensen TaraJensen added the requestor: NOAA/other NOAA Laboratory, not otherwise specified label Mar 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high High Priority requestor: METplus Team METplus Development Team requestor: NOAA/other NOAA Laboratory, not otherwise specified type: bug Fix something that is not working
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants