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

adds function to lookup and add station lat/lon to dataframe #1273

Merged
merged 3 commits into from
Jan 13, 2020

Conversation

kgoebber
Copy link
Collaborator

@kgoebber kgoebber commented Jan 7, 2020

Description Of Changes

This PR adds a simple function to use the station_info to add latitude and longitude values to a pandas.DataFrame, which will add plotting. This is useful for data obtained from sources that do not send lat/lon information with observations.

Checklist

  • Tests added
  • Fully documented

@kgoebber kgoebber requested a review from dopplershift as a code owner January 7, 2020 17:35
@dopplershift dopplershift added Area: IO Pertains to reading data Type: Feature New functionality labels Jan 9, 2020
@dopplershift dopplershift added this to the 1.0 milestone Jan 9, 2020
@kgoebber kgoebber force-pushed the add_station_lat_lon branch from 10c702c to a276ec4 Compare January 11, 2020 13:29
@@ -0,0 +1,24 @@
# Copyright (c) 2008,2015,2016,2017,2018,2019,2020 MetPy Developers.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should just be 2020.

dopplershift
dopplershift previously approved these changes Jan 12, 2020
Copy link
Member

@dopplershift dopplershift left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a rebase. If you want to fix the copyright year, great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: IO Pertains to reading data Type: Feature New functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants