Skip to content

Fix docs building#2155

Closed
iomaganaris wants to merge 1 commit into
masterfrom
magkanar/fix_docs_building
Closed

Fix docs building#2155
iomaganaris wants to merge 1 commit into
masterfrom
magkanar/fix_docs_building

Conversation

@iomaganaris

Copy link
Copy Markdown
Member

pip install docs/docs_requirements.txt brings now sphinx 6.1.3 where the API has changed and we should remove the RemovedInSphinx60Warning since it's no longer available

@alexsavulescu this failed locally but I'm wondering whether this fails in RTD as well

@alexsavulescu

Copy link
Copy Markdown
Member

@alexsavulescu this failed locally but I'm wondering whether this fails in RTD as well

We need to dig in a bit more. RTD still has 5.3.0 as default, we should check if enforcing 6.x works out (I will enable RTD builds for this branch).

Btw, for hocdomain.py we should rely on this: https://github.com/neuronsimulator/nrn/blob/master/docs/generate_hocdomain.py

@alexsavulescu

Copy link
Copy Markdown
Member

@alexsavulescu alexsavulescu marked this pull request as draft January 17, 2023 12:43
@azure-pipelines

Copy link
Copy Markdown

✔️ 430a636 -> Azure artifacts URL

@iomaganaris

iomaganaris commented Jan 17, 2023

Copy link
Copy Markdown
Member Author

Not sure why in the NEURON Documentation CI and RTD there are multiple and old versions of packages installed which are so old that they don't work with python 3.10-11 🤔

@alexsavulescu

Copy link
Copy Markdown
Member

Not sure why in the NEURON Documentation CI and RTD there are multiple and old versions of packages installed which are so old that they don't work with python 3.10-11 thinking

RTD uses miniconda whereas GH uses venv. Generally RTD follows along shortly after..

@iomaganaris

Copy link
Copy Markdown
Member Author

Not sure what was the issue in my local machine with this. It looks like RTD and the NEURON Documentation CI use the latest sphinx

@alexsavulescu alexsavulescu deleted the magkanar/fix_docs_building branch January 17, 2023 16:11
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