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

plotly graphs show in GitHub.io but not readthedocs #1044

Closed
ramcdougal opened this issue Mar 4, 2021 · 4 comments
Closed

plotly graphs show in GitHub.io but not readthedocs #1044

ramcdougal opened this issue Mar 4, 2021 · 4 comments

Comments

@ramcdougal
Copy link
Member

Graphs are visible and interactive here:

https://neuronsimulator.github.io/nrn/rxd-tutorials/Combining%20currents%20from%20mod%20files%20with%20rxd.html

but not on the corresponding readthedocs page:

https://nrn.readthedocs.io/en/latest/rxd-tutorials/Combining%20currents%20from%20mod%20files%20with%20rxd.html

They should work in both places.

@alexsavulescu
Copy link
Member

No indication as to why that didn't get through on RTD:

[NbConvertApp] Converting notebook Combining currents from mod files with rxd.ipynb to html
[NbConvertApp] Executing notebook with kernel: python3
Warning: no DISPLAY environment variable.
--No graphics will be displayed.
[NbConvertApp] Writing 4326458 bytes to Combining currents from mod files with rxd.html

@alexsavulescu
Copy link
Member

alexsavulescu commented Mar 4, 2021

From page logs:

Uncaught TypeError: require.undef is not a function
    at Combining currents from mod files with rxd.html:14791
(anonymous) @ Combining currents from mod files with rxd.html:14791

theme.js:3 Uncaught Error: Cannot find module 'plotly'
    at c (theme.js:3)
    at Combining currents from mod files with rxd.html:14873

@alexsavulescu
Copy link
Member

Looks related to : readthedocs/sphinx_rtd_theme#788

@alexsavulescu
Copy link
Member

So the issue was RTD was superseding sphinx_rtd_theme. Fixed in RTD via readthedocs/readthedocs.org#8059

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

Successfully merging a pull request may close this issue.

2 participants