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

TST: Travis CI working with wradlib 0.7.0 #493

Merged
merged 7 commits into from
Mar 2, 2016

Conversation

jjhelmus
Copy link
Contributor

@jjhelmus jjhelmus commented Mar 2, 2016

No description provided.

@kmuehlbauer
Copy link
Contributor

Just a reminder, the wradlib pypi version is not the latest version with the changes. You would need to clone from master.

@jjhelmus
Copy link
Contributor Author

jjhelmus commented Mar 2, 2016

@kmuehlbauer I plan on having Py-ART CI use the latest released version of wradlib not the bleeding edge. 0.7.0 is the latest release correct?

@kmuehlbauer
Copy link
Contributor

Correct! But python2/3 compatibility will be reached with 0.8.0.

@jjhelmus
Copy link
Contributor Author

jjhelmus commented Mar 2, 2016

I'll be limiting the tests to Python 2.7 at the moment.

@kmuehlbauer
Copy link
Contributor

@jjhelmus, it seems that wradlib stalls in travis-ci in its setup.py, because some dependency is missing. I suspect some gdal/netcdf issue. This will also be resolved with 0.8.0.

@jjhelmus
Copy link
Contributor Author

jjhelmus commented Mar 2, 2016

Yes, the issue is with gdal, the package provided by Continuum is finicky. I knew that this was failing just wanted a new CI run to debug. Should have a fix shortly.

@kmuehlbauer
Copy link
Contributor

Have a look here, this works for me: wradlib/wradlib#9

and this here wradlib/wradlib#12

@jjhelmus
Copy link
Contributor Author

jjhelmus commented Mar 2, 2016

The issue is that getting basemap and gdal to install correctly in a conda environment is difficult due to some of the packages incorrectly listing their dependencies, see ContinuumIO/anaconda-issues#584.

@kmuehlbauer
Copy link
Contributor

@jjhelmus ah, I see, so wradlib works because of basemap is no dependency. BTW, did you experience kind of connection problems with the conda-repos lately?

@jjhelmus
Copy link
Contributor Author

jjhelmus commented Mar 2, 2016

They seems a bit slow last night for me (~12 hours ago) but I haven't had any problems today.

@kmuehlbauer
Copy link
Contributor

OK, because I had some problems 5/6 days ago, where packages were not correctly installed within travis.

@jjhelmus
Copy link
Contributor Author

jjhelmus commented Mar 2, 2016

FYI, I created conda packages for wradlib 0.7.0 and uploaded them to by Anaconda.org channel. I'll be making a conda-forge pull request shortly for a more maintainable solution.

jjhelmus added a commit that referenced this pull request Mar 2, 2016
TST: Travis CI working with wradlib 0.7.0
@jjhelmus jjhelmus merged commit 2c6b515 into ARM-DOE:master Mar 2, 2016
@jjhelmus jjhelmus deleted the fix_wradlib_ci branch March 2, 2016 17:43
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.

2 participants