Skip to content

unified docs: one file for all languages#3466

Merged
ramcdougal merged 50 commits into
masterfrom
unified-docs
Aug 20, 2025
Merged

unified docs: one file for all languages#3466
ramcdougal merged 50 commits into
masterfrom
unified-docs

Conversation

@ramcdougal

@ramcdougal ramcdougal commented Jun 4, 2025

Copy link
Copy Markdown
Member

The idea here is to eliminate the duplication inherent in having a Python version, a HOC version, and possibly a MATLAB version of the documentation. The tabs are synced so you don't lose your language as you navigate or scroll.

Now when you search for e.g., SectionList, you'll only see one entry in the search results instead of two so you won't have to guess which is the language you want.

image
progref/modelspec/programmatic/topology/seclist.html

Downside: the help links would need some translation from NEURON 8.x (instead of python/ or hoc/ they all start with progref/)

The converted file is visible at https://nrn.readthedocs.io/en/unified-docs/progref/modelspec/programmatic/topology/seclist.html#

@azure-pipelines

Copy link
Copy Markdown

✔️ f4c639d -> Azure artifacts URL

This pulls the documentation from the right folder, but it doesn't filter for just the Python part.
@github-actions

github-actions Bot commented Jun 4, 2025

Copy link
Copy Markdown
Contributor

✔️ f4c639d -> artifacts URL

@github-actions

github-actions Bot commented Jun 4, 2025

Copy link
Copy Markdown
Contributor

✔️ e581171 -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ e581171 -> Azure artifacts URL

@codecov

codecov Bot commented Jun 4, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.42%. Comparing base (435a3db) to head (04db794).
⚠️ Report is 47 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3466      +/-   ##
==========================================
+ Coverage   68.37%   68.42%   +0.04%     
==========================================
  Files         684      684              
  Lines      116712   116716       +4     
==========================================
+ Hits        79803    79861      +58     
+ Misses      36909    36855      -54     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JCGoran

JCGoran commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator

Very nice idea! For reference, you can click on the readthedocs CI to access the built docs (https://nrn--3466.org.readthedocs.build/en/3466/), no need to manually activate branches on RTD.
For trying out locally, the following should work:

cmake -B build_docs -DNRN_ENABLE_DOCS=ON . -G Ninja
cmake --build build_docs --target sphinx

@ramcdougal

Copy link
Copy Markdown
Member Author

Thanks. I didn't realize the CI was always building the docs. Good to know!

@github-actions

github-actions Bot commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

✔️ b1ba54c -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ b1ba54c -> Azure artifacts URL

@github-actions

github-actions Bot commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

✔️ 2f43e99 -> artifacts URL

@github-actions

github-actions Bot commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

✔️ 850a777 -> artifacts URL

@github-actions

github-actions Bot commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

✔️ 7d48dd2 -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ 7d48dd2 -> Azure artifacts URL

@github-actions

github-actions Bot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

✔️ 81300ad -> artifacts URL

@github-actions

github-actions Bot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

✔️ e021d7b -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ e021d7b -> Azure artifacts URL

@github-actions

github-actions Bot commented Jun 7, 2025

Copy link
Copy Markdown
Contributor

✔️ ed0273f -> artifacts URL

@github-actions

Copy link
Copy Markdown
Contributor

✔️ 7bf9444 -> artifacts URL

@github-actions

Copy link
Copy Markdown
Contributor

✔️ ae546ef -> artifacts URL

@github-actions

Copy link
Copy Markdown
Contributor

✔️ 3dc9c03 -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ 3dc9c03 -> Azure artifacts URL

@github-actions

Copy link
Copy Markdown
Contributor

✔️ 2a97cd2 -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ 2a97cd2 -> Azure artifacts URL

@github-actions

Copy link
Copy Markdown
Contributor

✔️ feeee19 -> artifacts URL

@github-actions

Copy link
Copy Markdown
Contributor

✔️ a719820 -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ a719820 -> Azure artifacts URL

@ramcdougal ramcdougal marked this pull request as ready for review August 17, 2025 21:34
@azure-pipelines

Copy link
Copy Markdown

✔️ 6674d55 -> Azure artifacts URL

@github-actions

Copy link
Copy Markdown
Contributor

✔️ 6674d55 -> artifacts URL

@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

✔️ 04db794 -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ 04db794 -> Azure artifacts URL

@nrnhines nrnhines left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent. Let's merge and use this as the basis for future changes.

@ramcdougal ramcdougal merged commit 801ce19 into master Aug 20, 2025
43 checks passed
@ramcdougal ramcdougal deleted the unified-docs branch August 20, 2025 14:41
ramcdougal added a commit that referenced this pull request Aug 21, 2025
#3466 changed the URLs slightly
ramcdougal added a commit that referenced this pull request Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants