-
Notifications
You must be signed in to change notification settings - Fork 5
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
Dst update #57
Dst update #57
Conversation
Updated Dst tag to be informative, and easily allow for future additions of historic data from Kyoto.
I made some changes to partially address #59 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just made changes instead of requesting them. Tell me what you think @rstoneback
Fair enough. Good catch on the missing return. Adding a 'noaa' tag certainly makes it easy to extend to other data sources in the future. My only hesitation is it makes instantiating slightly harder but it does also make code written today more robust against future changes. Works for me. |
I have a draft pull over at pysatTutorials. My initial thinking was it could be good to have demos of the various pysatSpaceWeather instruments over there. Here is a simple example for Dst, Then I wondered how many we want to maintain, plus what is the balance against examples in the documentation. Topics for a telecom perhaps and outside this pull. Still, leaving this note here given the specific connection to dst. It is the reason I found the bug. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
Fixed the metadata re-assignment of the fill value.
Description
Addresses #56
Fixed multi-day data loading issue. Also improved efficiency when loading multiple days, plus, I added metadata.
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Test Configuration
Checklist:
develop
(notmain
) branchCHANGELOG.md
, summarizing the changesIf this is a release PR, replace the first item of the above checklist with the
release checklist on the pysat wiki:
https://github.com/pysat/pysat/wiki/Checklist-for-Release