-
Notifications
You must be signed in to change notification settings - Fork 75
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
IEM ASOS Data #254
base: main
Are you sure you want to change the base?
IEM ASOS Data #254
Conversation
…or IemAsos Class.
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.
Maybe this class lives in iastate.py
? Would using HTTPEndpoint
make things easier?
# Strip whitespace from the column names | ||
df.columns = df.columns.str.strip() | ||
|
||
df['valid'] = pd.to_datetime(df['valid'], format="%Y-%m-%d %H:%M:%S") |
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.
Would you mind adding df.units =
a dict of the units if possible here so it would work with our unit helper like we do with upper air data?
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'll take care of it!
from ..http_util import create_http_session | ||
|
||
|
||
class IemAsos: |
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.
Hmmm. IEMAsos
, 'IEMasos
, IAStateASOS
, IAStateasos
? Not sure on fighting convention for readability. Thoughts @dopplershift ?
Hi @WEP11, I've made some promises to do work on siphon + IEM web services and so I am reviewing things. Do you have interest in furthering this PR or can I "take it from here" ? :) |
@akrherz Go for it! I completely forgot about this, I must have had a busy 2018/2019. I deeply apologize for forgetting about this. But you're the IEM expert, so perhaps it's a blessing in disguise! |
This PR addresses #157 , adding the ability to download IEM ASOS data.
The IemAsos class will:
The DataFrame object is then accessible to the user through the data attribute, with column headers from IEM being retained for use by the DataFrame.
Still needed: