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

Peg Theano 1.0.5. #4032

Merged
merged 1 commit into from
Jul 28, 2020
Merged

Peg Theano 1.0.5. #4032

merged 1 commit into from
Jul 28, 2020

Conversation

twiecki
Copy link
Member

@twiecki twiecki commented Jul 28, 2020

No description provided.

@twiecki twiecki added this to the 3.9.3 milestone Jul 28, 2020
@twiecki
Copy link
Member Author

twiecki commented Jul 28, 2020

arviz compatiblity on travis is failing:


  - arviz

@AlexAndorra any idea?

@AlexAndorra
Copy link
Contributor

AlexAndorra commented Jul 28, 2020

It seems that it's failing when trying conda uninstall arviz -y, as if ArviZ wasn't on the environement in the first place: PackagesNotFoundError: The following packages are missing from the target environment.
Could it be related to @aseyboldt's pip-to-conda PR that you just merged? ArviZ was installed through pip before, IIRC.
In which case, deleting this line from travis.yml and adding it back in another PR (thus insuring that ArviZ was indeed installed and then uninstalled by conda) could do the trick?

@aseyboldt
Copy link
Member

@AlexAndorra That's probably the reason. Could be that this is due to caching. We use the same cache for all test subsets, so if we happen to use the cache of the wrong one, we might see an error like this.
I don't see why this wouldn't have been a problem before the conda switch though.
If this is true, we should be able to fix this by always re-installing arviz when we set up the env, even if it exists already.

@twiecki twiecki merged commit ee57bed into master Jul 28, 2020
@twiecki twiecki deleted the peg_theano_105 branch July 28, 2020 22:31
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.

3 participants