Skip to content

RTD failing yet again #3796

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

Closed
dcherian opened this issue Feb 24, 2020 · 7 comments
Closed

RTD failing yet again #3796

dcherian opened this issue Feb 24, 2020 · 7 comments

Comments

@dcherian
Copy link
Contributor

memory consumption errors as usual.

@keewis I remember you had an idea for using pip instead of conda?

@keewis
Copy link
Collaborator

keewis commented Feb 24, 2020

we could try moving everything except cartopy to the pip section in the environment file (iris is scitools-iris on PyPI). That seems to reduce the maximum memory consumption quite a bit (the conda-forge channel has grown almost too much).

@mathause
Copy link
Collaborator

This did indeed work for my project (regionmask), https://github.com/mathause/regionmask/blob/master/docs/environment_rtd.yml

@keewis
Copy link
Collaborator

keewis commented Feb 25, 2020

I tried it, but it still fails: https://readthedocs.org/projects/xray/builds/10502148/ We can't use pip directly since cartopy does not upload wheels and I'd expect the source install to fail.

If we look at the issue tracker of RTD, we can see a lot of issues regarding memory errors, so maybe this is a general issue with conda (when using conda-forge) and RTD.

Edit: seems I have to conda-install cf-units since otherwise it would need to be built on install.

@DocOtak
Copy link
Contributor

DocOtak commented Feb 25, 2020

The docs seem to build ok on Azure piplines, is it possible to get the built docs from that and publish somewhere? I do know this is possible with Travis, but haven't actually done it myself since my docs don't (yet?) have a memory problem.

@keewis
Copy link
Collaborator

keewis commented Feb 25, 2020

yeah, that would probably be the way forward. See #3697 for more discussion.

@dcherian
Copy link
Contributor Author

dcherian commented Mar 4, 2020

No luck with pinned versions and the stable build image: https://readthedocs.org/projects/xray/builds/10547287/

@keewis
Copy link
Collaborator

keewis commented Mar 13, 2020

due to the work of the RTD maintainers, our builds pass again. However, trying to build stable still fails, but instead of a memory error we get a sphinx failure: https://readthedocs.org/projects/xray/builds/10609593/

I think we already fixed that in master?

Also, we might want to switch back to the latest image in readthedocs.yml?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants