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

Add rasterio, gdal, python versions to the circleci build matrix #31

Open
perrygeo opened this issue Sep 21, 2016 · 3 comments
Open

Add rasterio, gdal, python versions to the circleci build matrix #31

perrygeo opened this issue Sep 21, 2016 · 3 comments

Comments

@perrygeo
Copy link
Contributor

Manually testing these permutations is a pain and it would be nice to ensure this (and other rio plugins) work across the board. Need to figure out the best way to make that happen with CircleCI.

@perrygeo
Copy link
Contributor Author

perrygeo commented Sep 21, 2016

I figured out how to test across python versions (2.7, 3.4, 3.5) on CircleCI with tox and pyenv. See https://github.com/mapbox/raster-tester/pull/32/files

However, the concept of a build matrix is a bit foreign to CircleCI. If we need to test multiple Rasterio and GDAL versions, we would be swimming against the current. That type of setup (all permutations of different versions) is relatively simple in Travis.

Do we need to test different GDAL and Rasterio versions? Many of us are testing GDAL 2 locally and we'll soon all be on the Rasterio 1.x train. So maybe sticking with CircleCI/tox/pyenv for python version testing is OK for now?

cc @dnomadb @virginiayung @sgillies

@virginiayung
Copy link

virginiayung commented Sep 21, 2016

Nice! @perrygeo I did try ☝️ the tox method here and it worked for the rio-alpha repo as well, but my problem in the end was with pyenv virtualenv not playing nicely with Matplotlib. Any suggestions with that? Thanks! Otherwise I'm happy with Travis :)

@sgillies
Copy link

@perrygeo let's let the Rasterio repo take all the responsibility for the GDAL dimension of the testing matrix. Raster-tester can then stick with one version. I'm pretty new to tox and not sure how one would use it to vary system libs.

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

No branches or pull requests

3 participants