Skip to content

Assorted improvements from #1929#2015

Merged
olupton merged 2 commits into
masterfrom
olupton/assorted-improvements
Oct 10, 2022
Merged

Assorted improvements from #1929#2015
olupton merged 2 commits into
masterfrom
olupton/assorted-improvements

Conversation

@olupton

@olupton olupton commented Oct 7, 2022

Copy link
Copy Markdown
Collaborator

Various incidental improvements from #1929.

  • STETransition et al. now use STL components
  • Removal of #if 0 block in nrn_finitialize fixes formatting there
  • table_check_ no longer uses Datum
  • PyHocObject is consistently defined in only one place
  • Exception messages are printed to stderr when they are caught
  • example_nmodl tests are unpacked into different tests, Python versions are run (not just HOC)
  • "Test fast_imem calculation" tests now check status codes

@olupton olupton marked this pull request as ready for review October 7, 2022 12:38
@olupton olupton requested review from alkino and pramodk October 7, 2022 13:01
@azure-pipelines

Copy link
Copy Markdown

✔️ 27bf71d506569bbab5eb0800502cf09422e66516 -> Azure artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ de5169c78b82802abfe043bd2d85986fd217d119 -> Azure artifacts URL

@codecov-commenter

codecov-commenter commented Oct 7, 2022

Copy link
Copy Markdown

Codecov Report

Merging #2015 (3f6b2c3) into master (39e36db) will decrease coverage by 0.00%.
The diff coverage is 95.12%.

@@            Coverage Diff             @@
##           master    #2015      +/-   ##
==========================================
- Coverage   47.51%   47.51%   -0.01%     
==========================================
  Files         528      528              
  Lines      118852   118802      -50     
==========================================
- Hits        56468    56444      -24     
+ Misses      62384    62358      -26     
Impacted Files Coverage Δ
src/nrncvode/netcon.h 53.48% <ø> (ø)
src/nrnpython/grids.h 100.00% <ø> (ø)
src/nrnpython/nrnpy_hoc.cpp 66.86% <ø> (+0.37%) ⬆️
src/oc/hoc.cpp 71.72% <50.00%> (-0.07%) ⬇️
src/nrnoc/fadvance.cpp 77.94% <93.10%> (ø)
src/nrncvode/netcvode.cpp 88.50% <100.00%> (+0.01%) ⬆️
src/nrniv/nrnste.cpp 98.82% <100.00%> (-0.35%) ⬇️
src/nrniv/nrnste.h 100.00% <100.00%> (ø)
src/nrnoc/multicore.cpp 90.33% <100.00%> (+0.10%) ⬆️
test/unit_tests/unit_test.cpp 100.00% <100.00%> (ø)
... and 6 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@azure-pipelines

Copy link
Copy Markdown

✔️ 2cf08fea077eb7aa5336caa40f198da794cd8462 -> Azure artifacts URL

@olupton olupton force-pushed the olupton/assorted-improvements branch from 2cf08fe to b679f16 Compare October 8, 2022 05:57
@azure-pipelines

Copy link
Copy Markdown

✔️ b679f16 -> Azure artifacts URL

@alexsavulescu

Copy link
Copy Markdown
Member

will decrease coverage by 0.04%.

Not inflicted by this PR. Related to #1663

@pramodk pramodk 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.

a few minor questions for clarity otherwise looks good to me!

Comment thread src/nrniv/nrnste.h Outdated
Comment thread test/CMakeLists.txt Outdated
Comment thread src/nrnpython/nrn_pyhocobject.h Outdated
Comment thread src/nrniv/nrnste.h
Comment thread src/nrniv/nrnste.h
@azure-pipelines

Copy link
Copy Markdown

✔️ 3f6b2c3 -> Azure artifacts URL

@olupton olupton merged commit 67f9bd2 into master Oct 10, 2022
@olupton olupton deleted the olupton/assorted-improvements branch October 10, 2022 11:34
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.

4 participants