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.

Sign up for GitHub

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

Jump to bottom

Add Key to discus #80

Closed
timtrice opened this issue Jun 24, 2017 · 0 comments
Closed

Add Key to discus #80

timtrice opened this issue Jun 24, 2017 · 0 comments
Labels
High Priority NOW NOW NOW

Comments

@timtrice
Copy link
Contributor

In 2006, Key became part of the header for storm discussions (discus). Add this variable and update the dataset.
@timtrice timtrice added the High Priority NOW NOW NOW label Jun 24, 2017
@timtrice timtrice added the Review Under review, analysis label Jun 24, 2017
timtrice added a commit that referenced this issue Jun 24, 2017
@timtrice
Add var Key to discus dataset (#80)
11bef08 
* Added variable `key` to `discus` dataframe. `key` was added to storm
discussions beginning 2006 season. For cyclones prior, `key` will be
NA.

* Updated test datasets. All tests pass.

* All checks pass.
@timtrice timtrice removed the Review Under review, analysis label Jun 24, 2017
timtrice added a commit that referenced this issue Jun 24, 2017
@timtrice
Add var Key to discus dataset (#80)
b597648 
* Added variable `key` to `discus` dataframe. `key` was added to storm
discussions beginning 2006 season. For cyclones prior, `key` will be
NA.

* Updated test datasets. All tests pass.

* All checks pass.
timtrice added a commit that referenced this issue Jun 26, 2017
@timtrice
Modify, clean up return dataframes
2d539ca 
* `Key` variable added to `discus` dataframes. `Key` will be NA for all cyclones >= 2005. Should not be <= 2006. (#80)

* Removed `Adv` variable from `posest` dataframes. Position estimates do not have advisory numbers. (#81)

* Removed `Adv` variable from `update`. Updates do not have advisory numbers. (#84)

* Added variable `Key` to `get_public` dataframes. (#85)

* Added variable `Key` to `get_update` dataframes. (#86)

* Removed non-existent wind radii variables in `get_fstadv`. Hrs 48 and 72 hours only have 34 and 50kt wind fields. Hrs 96 and 120 have none. (#89)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
High Priority NOW NOW NOW
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@timtrice