Skip to content

Conversation

@Titan-C
Copy link
Member

@Titan-C Titan-C commented Dec 21, 2016

Shamelessly copy from MNE-python Circle CI script not to install conda again

@lesteve
Copy link
Member

lesteve commented Dec 21, 2016

What problem does this solve?

@Titan-C
Copy link
Member Author

Titan-C commented Dec 21, 2016

What problem does this solve?

Build on the same branch or PR start using the cache and then fail when conda tries to install. See

https://circleci.com/gh/Titan-C/sphinx-gallery/11

@lesteve
Copy link
Member

lesteve commented Dec 21, 2016

I see, they are caching the miniconda install. Not sure I am a big fan of the idea, I'd rather remove ~/miniconda from the cache.

@Titan-C
Copy link
Member Author

Titan-C commented Dec 21, 2016

I see, they are caching the miniconda install. Not sure I am a big fan of the idea, I'd rather remove ~/miniconda from the cache.

I see the cache is because one downloads a lot. Comparing timings for us downloading conda with mayavi and mkl + install takes a little more than a minute. For MNE-Python they need 3 min to recover their cache which includes more than the conda install. I'm fine with dropping the cache

@Titan-C
Copy link
Member Author

Titan-C commented Dec 21, 2016

It's a bit weird, but building without saving cache breaks the build because cache was already there. I'm rebuilding from CircleCi to clean everything

@lesteve
Copy link
Member

lesteve commented Dec 21, 2016

Let's merge this one thanks!

@lesteve lesteve merged commit f3ccb80 into sphinx-gallery:master Dec 21, 2016
@Titan-C Titan-C deleted the circle_cache branch December 22, 2016 14:56
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