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

ReadTheDocs not rendering notebook after a recent PR #1305

Closed
willingc opened this issue Apr 6, 2016 · 11 comments · Fixed by #1306
Closed

ReadTheDocs not rendering notebook after a recent PR #1305

willingc opened this issue Apr 6, 2016 · 11 comments · Fixed by #1306

Comments

@willingc
Copy link
Member

willingc commented Apr 6, 2016

@takluyver Would you mind checking if it is rendering on your system?

screen shot 2016-04-05 at 6 36 26 pm

@willingc
Copy link
Member Author

willingc commented Apr 6, 2016

Checking if PR #1266 or #1304 introduced a new issue.

@willingc
Copy link
Member Author

willingc commented Apr 6, 2016

Looks like the CSS changes (two commits: b3788c2 and 9c56f45) in #1266 is the cause of RTD to display as above. The commits build fine locally and pass as green on RTD yet the rendering is incorrect. I will modify if I can in a new PR within an hour; otherwise, I will revert #1266

@willingc willingc added the bug label Apr 6, 2016
@willingc willingc self-assigned this Apr 6, 2016
willingc added a commit that referenced this issue Apr 6, 2016
Remove custom.css added in #1266
Closes #1305
@willingc
Copy link
Member Author

willingc commented Apr 6, 2016

@mgeier I'm not sure why the custom CSS did not render properly on RTD. Thanks for your work on PR #1266. I only backed out the code that was giving RTD trouble.

@mgeier
Copy link
Contributor

mgeier commented Apr 6, 2016

I suspect that this is actually a RTD bug: readthedocs/readthedocs.org#2116.

@willingc
Copy link
Member Author

willingc commented Apr 6, 2016

@mgeier I agree. Thanks for filing the issue on RTD. 🍰

@minrk minrk added this to the not notebook milestone Apr 8, 2016
@mgeier
Copy link
Contributor

mgeier commented Apr 12, 2016

@willingc There was a response at readthedocs/readthedocs.org#2116, but I don't like the proposed solution (because it is IMHO a work-around and no real solution).
I also asked at Sphinx: sphinx-doc/sphinx#2442

Any opinions on that?

@mgeier
Copy link
Contributor

mgeier commented Apr 27, 2016

@willingc Did you have an opportunity to look into this?

I could really need somebody to support me on readthedocs/readthedocs.org#2116!

@mgeier
Copy link
Contributor

mgeier commented May 14, 2016

@willingc FYI: the problem solved itself because I included the CSS fixes directly into nbsphinx.

I still think the original problem is a bug in RTD and Sphinx ...

@willingc
Copy link
Member Author

@mgeier Thanks for the update. I will see some of the RTD folks in 2 weeks. I'll see if I can have them look at in person.

@mgeier
Copy link
Contributor

mgeier commented Jun 6, 2016

@willingc Did you have a chance to talk to the RTD people?
I don't know what to answer to readthedocs/readthedocs.org#2116 (comment), I think I've already said everything ...

Currently, I'm discussing this on a Sphinx issue, see sphinx-doc/sphinx#2442 (comment).

It would really be nice to get your opinion on this (be it for or against my statements).

@willingc
Copy link
Member Author

willingc commented Jun 6, 2016

@mgeier Traveling today. Just left some comments here: sphinx-doc/sphinx#2442 (comment)

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants