search

ioos / system-test

IOOS DMAC System Integration Test project
github.com/ioos/system-test/wiki
The Unlicense
7 stars 14 forks source link

Fix travis ci #202

Closed ocefpaf closed 9 years ago

ocefpaf commented 9 years ago

This PR adds missing conda-requirements.txt, pip-requirements.txt and .py files so travis-ci is OK.

ocefpaf commented 9 years ago

Now travis-ci results finally have some meaning.

jkupiec commented 9 years ago

Is this ready for a Merge?

ocefpaf commented 9 years ago

Can @Bobfrat or @birdage review and merge?

birdage commented 9 years ago

@ocefpaf i can look at this tomorrow

Bobfrat commented 9 years ago

@ocefpaf Looks good! I'm just checking that the conda requirement files works for each of the notebooks. I'll let you know in a bit.

Bobfrat commented 9 years ago

@ocefpaf, I'm having trouble with geojson in the conda-requirements file for Scenario_3A_Important_Bird_Areas. Is this an OSX issue?

conda install geojson
Fetching package metadata: ....
Error: No packages found in current osx-64 channels matching: geojson
ocefpaf commented 9 years ago

There is no geojson for OSX in @rsignell-usgs binstar channel.

Maybe it is time to open an issue at https://github.com/ioos/conda-recipes asking for a OSX binaries curator.

Bobfrat commented 9 years ago

Ok I couldn't conda install folium or rdflib dependencies either.

Error: No packages found in current osx-64 channels matching: folium ==0.1.2_dev_f7194ad
Error: Could not find some dependencies for rdflib: isodate
ocefpaf commented 9 years ago

@Bobfrat the fact that those packages do not have a OSX version in @rsignell-usgs binstar channel is beyond the scope of this PR.

Lets just open an issue at https://github.com/ioos/conda-recipes for the missing OSX binaries later.

We can merge this PR as soon as you give me the green on your review. The important part is to check if I covered all the notebooks and imports inside them. No need to run all the notebooks, just use a grep import for example.

Bobfrat commented 9 years ago

:+1: This pull request is all set.