Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
Ryan May
@dopplershift
sounds good!
Warren Pettee
@WEP11
@dopplershift My unit test appears to be passing on my end. Do I only need to commit my test's data from the fixtures folder?
Ryan May
@dopplershift
Looks like you got what you need, the test worked fine. It’s picking up a few style issues, though: https://ci.appveyor.com/project/Unidata/siphon/build/1.0.265/job/kax58faoy5kv1m26/tests
Sorry, I was unclear on the quoting, that’s my bad. We use single quotes throughout…except docstrings, which every tool seems to insist on having as ”””.
Warren Pettee
@WEP11
@dopplershift It looks like I finally beat the bots! Let me know if you spot any more issues or need to see any changes!
Ryan May
@dopplershift
@WEP11 Will do. Thanks for working through this with us?
John Leeman
@jrleeman
So I was cleaning up #176 - any opposition to a dictionary for the regions that are valid mapping to keys of their "human" names (i.e. North America)? It lets us check that the region is valid and not ask people to find obscure abbreviations.
Ryan May
@dopplershift
Nope, that’s fine. Any reason you haven’t merged #197?
Oh, I guess you can’t. :sheep: Can you review?
John Leeman
@jrleeman
Right... Yeah, let me finish up writing the test I'm on and I'll review... Why was Appveyor failing?
Ryan May
@dopplershift
Because we have files without a copyright from the ACIS PR. Stupid flake8-copyright plugin.
John Leeman
@jrleeman
I'm getting a doc warning from ACIS as well locally
I've fixed it in my branch so I can render docs locally
In basic overview a space was missing from # * Note: Missing data is recorded as M!
(The one before the N) Made it think it was italics with no closing
Ryan May
@dopplershift
interesting
John Leeman
@jrleeman
Should we lowercase whatever the user passes for region and use that as keys so North America, NORTH AMERICA, and north america all work?
Ryan May
@dopplershift
sure
John Leeman
@jrleeman
Wyoming soundings have station lat/lon/elevation in the footer. IA does not. Should we include it for WY or not since both don't have it?
Ryan May
@dopplershift
:scream:
John Leeman
@jrleeman
I'd opt for reporting it - if someone is using multiple sources it could be handled by checking for the column. Just seems like really useful data
Speaking from experience on the CAPE project
Ryan May
@dopplershift
We don’t have one interface to paper over them, so I guess it’s not a big deal for them to differ.
John Leeman
@jrleeman
I also need to email larry that the region selector on the API doesn't actually do anything. You can request OUN with a region of Antarctica and it's happy to return the right data
Ryan May
@dopplershift
lol
Sean Arms
@lesserwhirls
45c11 d474 rul35 lolololol
Andrew
@IAteAnDrew1_twitter
I have a question about the examples page. I'm a bit confused about what I can do with this example after I get the list of catalog_refs
image.png
John Leeman
@jrleeman
You can drill further down or grab datasets. Have a look at some of the gallery examples: http://unidata.github.io/python-gallery/index.html or the workshop material on siphon https://github.com/Unidata/unidata-python-workshop/tree/master/notebooks/Siphon
Andrew
@IAteAnDrew1_twitter
Thanks! I feel like the workshop material should be linked on the documentation (I might have missed it though!)
John Leeman
@jrleeman
@IAteAnDrew1_twitter That's an excellent point - would you mind opening an issue on siphon for that? We'll make sure it happens!
Andrew
@IAteAnDrew1_twitter
yep, just did it. thanks!
Scott
@scollis
Hey all! Rather than add to your issue tracker I thought I would check an issue I am having here ...
Some times I get this error requests.exceptions.HTTPError: Error accessing http://thredds.ucar.edu/thredds/ncss/nws/metar/ncdecoded/Metar_Station_Data_fc.cdmr?var=weather&var=dew_point_temperature&var=inches_ALTIM&var=air_temperature&var=wind_from_direction&var=cloud_area_fraction&var=wind_speed&time=2018-06-11T15%3A17%3A59&west=-87.0&east=-89.0&south=40.5&north=42.7&accept=csv: 500 Infinite loop in linked list at recno= 23284
Note I have now wrapped it in a try to get it working again.. but no pretty obs
Is it a server issue?
Ryan May
@dopplershift
Yeah, it’s a pretty normal issue with the surface obs. Somebody thought it was a good idea to implement a linked list within a netcdf file, which means things go haywire occasionally. We need to replace it.
Scott
@scollis
Cool.. so not part of Siphon? Let me know if I can help..
dafekt1ve
@dafekt1ve
@jrleeman Thanks for letting me know about the IAStateUpperAir data pull update. When do these updates hit anaconda? Is there a nightly build?
Ryan May
@dopplershift
No nightly build. If you can manage a git checkout (or even just download the zip of the repo from GitHub), you can install it with a pip install -e . from the checkout.
dafekt1ve
@dafekt1ve
Thanks @dopplershift. Got it and I am trying to test it out for a case I am looking at. You had mentioned to me at the Users Workshop that there was a lookup table for the sounding sites to get their lat-long if you knew the 3 or 4-letter code. Can you point me to it?
Ryan May
@dopplershift
I did?
dafekt1ve
@dafekt1ve
Hehe... I thought you did. Maybe it was one of the Johns instead. I am using the IA State Upper Air pull and it doesn't have the lat-long points like the University of Wyoming dataset does, so I need to find them somewhere. You have an idea of how/where to do that?
Ryan May
@dopplershift
There are some lists of stations here: https://github.com/Unidata/station-lists But no code to easily do what you’re looking for.
Maybe @jrleeman or @jthielen know something else
John Leeman
@jrleeman
I've got a station list in the CAPE work I did that might be helpful: https://github.com/jrleeman/CAPE-SciPy-2017/tree/master/Notebooks
Bryan Guarente
@bryanguarente
Thanks @jrleeman That'll work nicely. Is something like this lookup table worth putting into the codebase or building a class/function for doing this? Seems like something that will have to be done often now with the new IAStateUpperAir method.
John Leeman
@jrleeman
I've thought about including it... but (there's always one) we need to encode time as well since multiple stations have been relocated throughout history. Probably something to go data hunting for at some point.
Andrew
@IAteAnDrew1_twitter
is this related?
https://mesonet.agron.iastate.edu/sites/networks.php?network=RAOB&format=csv&nohtml=on
if so, you can do pd.read_csv('https://mesonet.agron.iastate.edu/sites/networks.php?network=RAOB&format=csv&nohtml=on')