mesowx / MesoPy

A Python wrapper for the MesoWest environmental data API
MIT License
58 stars 23 forks source link

Catch error for long (2+ year) date requests #22

Closed dnowacki-usgs closed 7 years ago

dnowacki-usgs commented 7 years ago

This PR catches JSON errors, often caused by long date range requests. See #21

joeyoun9 commented 7 years ago

Just commenting Re the travis CI failure, I think that is just woefully misconfigured, not that this simple update is problematic.

adamabernathy commented 7 years ago

I agree wi h Joe. I can fix the tests and the config file to get that going again tomorrow.

Sent from my iPhone

On Aug 20, 2017, at 11:23 AM, Joe Young notifications@github.com wrote:

Just commenting Re the travis CI failure, I think that is just woefully misconfigured, not that this simple update is problematic.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

adamabernathy commented 7 years ago

Ok, pending review from @joeyoun9 for PR #23 and merge into the repo, we should then re-run the tests for this PR #22 and hopefully make the merge later today.