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

IEM ASOS Data #254

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

WEP11
Copy link
Contributor

@WEP11 WEP11 commented Oct 26, 2018

This PR addresses #157 , adding the ability to download IEM ASOS data.

The IemAsos class will:

  1. Collect a list of station ID's, a start, and an end date from the user
  2. Build a URL for IEM's asos.py
  3. Collect the data into a Pandas DataFrame

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:

  • Examples

Copy link
Contributor

@jrleeman jrleeman left a 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")
Copy link
Contributor

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?

Copy link
Contributor Author

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:
Copy link
Contributor

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 ?

@akrherz
Copy link
Contributor

akrherz commented Nov 17, 2021

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" ? :)

@WEP11
Copy link
Contributor Author

WEP11 commented Nov 17, 2021

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants