Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

CLIMATE-875 Upgrade to Podaacpy 1.4.0 #415

Merged
merged 1 commit into from Oct 21, 2016
Merged

Conversation

lewismc
Copy link
Member

@lewismc lewismc commented Oct 21, 2016

@OCWJenkins
Copy link

Merged build triggered. Test Failed.

@lewismc
Copy link
Member Author

lewismc commented Oct 21, 2016

OK folks, so this PR passes against Python 2.7... which is great. Thank you @agoodm for stabilizing and significantly simplifying our builds and packaging.
There are a number of issue here with Python 3.4 and 3.5 which we will address over on additional tickets. Note, that the podaac_datasource tests actually pass fine on subsequent versions of Python.
If anyone could review this clean PR it would be greatly appreciated.
Ta

@agoodm
Copy link
Member

agoodm commented Oct 21, 2016

I ran the example, LGTM.

Regarding the python 3 build issues most of the those errors are pretty trivial to fix, so I'll see if I can get around to doing that by the end of the week. The only thing that might be difficult is rcmed since it uses urllib. We might have to learn by example from podaacpy and add future as a dependency here.

@asfgit asfgit merged commit 088706d into apache:master Oct 21, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants