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

Simplify CI setup and environment #3056

Merged
merged 6 commits into from
Oct 7, 2018
Merged

Simplify CI setup and environment #3056

merged 6 commits into from
Oct 7, 2018

Conversation

philippjfr
Copy link
Member

No description provided.

@stonebig
Copy link
Contributor

stonebig commented Oct 6, 2018

pillow-5.3.0 is not ok ?

@philippjfr
Copy link
Member Author

We're seeing some issues with matplotlib gif rendering. For now this is just a test if this is really the cause.

@philippjfr
Copy link
Member Author

Doesn't appear to be the issue.

@philippjfr
Copy link
Member Author

The issue is probably an change in matplotlib 2.2.3 in that case.

@philippjfr philippjfr force-pushed the pin_pillow branch 6 times, most recently from 1a5678d to cbd46ed Compare October 7, 2018 00:38
@philippjfr
Copy link
Member Author

philippjfr commented Oct 7, 2018

@jbednar @jlstevens I haven't been able to track down precisely what the cause of the gif export error was, pinning matplotlib and pillow to older versions individually did not work but now I'm pinning both of them it is working again. As such I'm going to merge this PR soon, but I'll quickly describe what I've done.

This PR is intended as a stepping stone towards slowly moving to @ceball's fabulous new architecture. I've moved all the dependencies out of the .travis.yml into the environment.yml which has long been unused. The environment.yml is much more readable and makes it clear just how much we're pinning right now. We should try to slowly lift the pinned versions while keeping the tests passing. Once we can have our testing setup work on the latest releases the transition to the new architecture should be much easier and cleaner.

One thing to note here is that the environment.yml does not define a python version, which means that at least on travis the only way we are getting both py2 and py3 tests is that we are installing Miniconda and Miniconda3, which then determines the python version of the environment while the exact TRAVIS_PYTHON_VERSION env var is not respected. This should be safe in the near term but it is only a temporary solution until we switch over to the new setup.

@philippjfr philippjfr changed the title Pinned pillow version Simplify CI setup and environment Oct 7, 2018
@jlstevens
Copy link
Contributor

Just a quick comment to say I am happy with the changes Philipp made in this PR: we want to minimize travis.yml as much as possible!

philippjfr added a commit that referenced this pull request Oct 25, 2018
# Conflicts:
#	.travis.yml
@philippjfr philippjfr deleted the pin_pillow branch November 12, 2018 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants