Skip to content

documentation updated to use "from neuron import n"#3459

Merged
ramcdougal merged 39 commits into
masterfrom
from-neuron-import-n
Jun 4, 2025
Merged

documentation updated to use "from neuron import n"#3459
ramcdougal merged 39 commits into
masterfrom
from-neuron-import-n

Conversation

@ramcdougal

@ramcdougal ramcdougal commented Jun 2, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

i.e., updated imports, n. instead of h. and n( instead of h(, try to avoid interferring with existing n variables

Kept the nrnpython("print(h.tup)") in docs/hoc/programming/python-from-hoc.rst because I think using h there and in the line above makes more sense than n.

ramcdougal added 28 commits June 1, 2025 06:20
@ramcdougal
updated to 'from neuron import n'
838e6ba
@ramcdougal
n.Section
9f4fd51
@ramcdougal
Julia example
33bbcd1
@ramcdougal
finitialize, continuerun, continue_dialog
0703ab9
@ramcdougal
IClamp, Vector, FInitializeHandler
1bd4d60
@ramcdougal
loadfile, xopen, cvode*, CVode
dd811e2
@ramcdougal
PlotShape, Graph
87a7303
@ramcdougal
Matrix, List, NetCon, Parallel*, Mechanism*
1948eb0
@ramcdougal
axon, soma, dend, apic, pas, hh
8ca2229
@ramcdougal
h._ref_ but not hh._ref_
162d142
@ramcdougal
dt, steps_per_ms
92efc8d
@ramcdougal
xlabel, xvarlabel, xpanel, Deck, HBox, VBox
b65a942
@ramcdougal
topology, SaveState, BBSaveState, some h.t
ecb4392
@ramcdougal
fadvance, advance, step
5140bf1
@ramcdougal
gui things, section things, remaining classes
bf49323
@ramcdougal
fixed a Grapn, many of the easy functions
0a84d20
@ramcdougal
oldgrph
000100f
@ramcdougal
internals
487c57a
@ramcdougal
missed an h( in internals
0be89e9
@ramcdougal
direct calls of HOC via h( now use n(
516d3f7
@ramcdougal
fixed a for n which would overwrite NEURON's n
8f61b63
@ramcdougal
fixed uses of n as a non-NEURON variable
5b2a46f
@ramcdougal
the remaining h.a* and h.b*
fafea1a
@ramcdougal
remaining h.c* - h.f*
92accd5
@ramcdougal
remaining h.g* - h.l*
dc7c70e
@ramcdougal
h.m*-h.o*
2068a5e
@ramcdougal
Finished the alphabetical search
546f828
@ramcdougal
Name for n isn't the same as for h nor was either what was there before
666fdb6
@ramcdougal ramcdougal requested review from noahsafar and nrnhines June 2, 2025 00:43
@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

✔️ 3b813bf -> artifacts URL

@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

✔️ 0050486 -> artifacts URL

nrnhines
nrnhines reviewed Jun 2, 2025
View reviewed changes
Comment thread docs/rxd-tutorials/circadian.ipynb
@azure-pipelines

azure-pipelines Bot commented Jun 2, 2025

Copy link
Copy Markdown

✔️ 0050486 -> Azure artifacts URL

@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

✔️ d5d6707 -> artifacts URL

@codecov

codecov Bot commented Jun 2, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.38%. Comparing base (c10ab51) to head (01ed3dd).
⚠️ Report is 63 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #3459      +/-   ##
==========================================
+ Coverage   68.37%   68.38%   +0.01%     
==========================================
  Files         682      682              
  Lines      116582   116582              
==========================================
+ Hits        79713    79725      +12     
+ Misses      36869    36857      -12

☔ 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.

@azure-pipelines

azure-pipelines Bot commented Jun 2, 2025

Copy link
Copy Markdown

✔️ d5d6707 -> Azure artifacts URL

@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

✔️ 7f09309 -> artifacts URL

@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

✔️ 083d58f -> artifacts URL

@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

✔️ fe1060d -> artifacts URL

@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

✔️ 5413b32 -> artifacts URL

@azure-pipelines

azure-pipelines Bot commented Jun 2, 2025

Copy link
Copy Markdown

✔️ 5413b32 -> Azure artifacts URL

noahsafar
noahsafar reviewed Jun 2, 2025
View reviewed changes
Comment thread docs/changelog.md
Comment thread docs/courses/code/hh_patch.py Outdated
Comment thread docs/courses/code/initplotfi.py Outdated
Comment thread docs/hoc/programming/python-from-hoc.rst
@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

✔️ 01ed3dd -> artifacts URL

@github-actions

github-actions Bot commented Jun 2, 2025

Copy link
Copy Markdown
Contributor

✔️ f173f79 -> artifacts URL

@sonarqubecloud

sonarqubecloud Bot commented Jun 2, 2025

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@azure-pipelines

azure-pipelines Bot commented Jun 2, 2025

Copy link
Copy Markdown

✔️ 01ed3dd -> Azure artifacts URL

@ramcdougal ramcdougal requested a review from noahsafar June 3, 2025 02:35
@ramcdougal ramcdougal merged commit 00d603a into master Jun 4, 2025
43 checks passed
@ramcdougal ramcdougal deleted the from-neuron-import-n branch June 4, 2025 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nrnhines nrnhines nrnhines approved these changes
@noahsafar noahsafar noahsafar approved these changes

Assignees

No one assigned

Labels

documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ramcdougal @nrnhines @noahsafar