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

NEURON course exercise sets #1735

Merged
merged 32 commits into from
May 13, 2022
Merged

NEURON course exercise sets #1735

merged 32 commits into from
May 13, 2022

Conversation

ramcdougal
Copy link
Member

@ramcdougal ramcdougal commented Mar 25, 2022

18 of the exercise sets from the 2018 NEURON course. The neuron.yale.edu site only has 16 of these. Our extra ones are numbers 9 (inhomogeneous channel distributions) and 11 (the linear circuit builder).

Partial progress toward #1728 and this will resolve #1685

http://nrn.readthedocs.io/en/docs-update/courses/exercises2018.html

@codecov-commenter
Copy link

codecov-commenter commented Mar 25, 2022

Codecov Report

Merging #1735 (f6fc610) into master (3f2f83d) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1735      +/-   ##
==========================================
+ Coverage   45.45%   45.49%   +0.03%     
==========================================
  Files         550      550              
  Lines      113130   113130              
==========================================
+ Hits        51423    51468      +45     
+ Misses      61707    61662      -45     
Impacted Files Coverage Δ
src/parallel/bbs.cpp 76.27% <0.00%> (+1.69%) ⬆️
src/parallel/bbssrvmpi.cpp 47.56% <0.00%> (+6.09%) ⬆️
src/parallel/bbsclimpi.cpp 59.19% <0.00%> (+8.62%) ⬆️
src/nrnmpi/bbsmpipack.cpp 92.19% <0.00%> (+10.24%) ⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

…llbuilder_oveview_and_hints but was unable to git add it. had issues adding images to the second file.
…using morphometric data exercise page and still working on it
…plementary links to pages, not finished with managing models on the fly but almost done
@ramcdougal ramcdougal marked this pull request as ready for review May 10, 2022 16:22
@ramcdougal ramcdougal requested review from adamjhn and Helveg May 11, 2022 17:28
Copy link
Member

@adamjhn adamjhn left a comment

Choose a reason for hiding this comment

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

It's a good copy of the current course exercises on the NEURON website. I've made a few minor suggestions.

docs/courses/using_morphometric_data.rst Outdated Show resolved Hide resolved
docs/courses/using_morphometric_data.rst Outdated Show resolved Hide resolved
docs/courses/using_morphometric_data.rst Outdated Show resolved Hide resolved
docs/courses/using_nmodl_files.rst Outdated Show resolved Hide resolved
docs/courses/using_nmodl_files.rst Outdated Show resolved Hide resolved
docs/courses/rxd_exercises.rst Outdated Show resolved Hide resolved
docs/courses/hints_for_inhomogeneous_channel_dist.rst Outdated Show resolved Hide resolved
docs/courses/hints_for_inhomogeneous_channel_dist.rst Outdated Show resolved Hide resolved
docs/courses/batch_runs_bulletin_board_parallelization.rst Outdated Show resolved Hide resolved
docs/courses/batch_runs_bulletin_board_parallelization.rst Outdated Show resolved Hide resolved
@ramcdougal ramcdougal merged commit 9455cf7 into master May 13, 2022
@alexsavulescu alexsavulescu mentioned this pull request Jun 29, 2022
19 tasks
@alexsavulescu alexsavulescu deleted the docs-update branch July 20, 2022 15:14
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.

Course exercises should be migrated from NEURON website to readthedocs
4 participants