-
Notifications
You must be signed in to change notification settings - Fork 113
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
Partrans cover #1051
Partrans cover #1051
Conversation
more convenient for more complicated messages for hoc_execerror(const char*, const char*) Avoid issues when creating 0 length arrays in pc.setup_transfer().
Message truncates after 512 bytes. fileio.cpp no -Wall warnings.
Experiencing two problems: 1) threads 2) voltage source with extracellular inserted and two targets for it.
Enabled by '-DNRN_ENABLE_COVERAGE=ON' (default, coverage of everything). -DNRN_COVERAGE_FILES='semicolon;list;of;files;to;cover' The files are relative to PROJECT_SOURCE_DIR, e.g.src/nrniv/partrans.cpp and, for now, can only be the sources involved in the nrniv_lib target. Provides: 'make cover_begin' to initialize coverage. 'make cover_html' generate an html report with all coverage since last 'make cover_begin'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice !
# Specific coverage files: | ||
# - OFF : Disable code coverage. | ||
# - ON : Collect code coverage for files (default all). | ||
# - <string> : semicolon (;) separated list of files to collect coverage. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like the idea 🚀
Co-authored-by: Alexandru Săvulescu <alexandru.savulescu@epfl.ch>
The consequence of fewer functions increases function coverage from 79% to 93%.
Codecov Report
@@ Coverage Diff @@
## master #1051 +/- ##
==========================================
+ Coverage 29.29% 30.57% +1.28%
==========================================
Files 570 570
Lines 108683 108666 -17
==========================================
+ Hits 31835 33227 +1392
+ Misses 76848 75439 -1409
Continue to review full report at Codecov.
|
Co-authored-by: Alexandru Săvulescu <alexandru.savulescu@epfl.ch>
Co-authored-by: Alexandru Săvulescu <alexandru.savulescu@epfl.ch>
Co-authored-by: Alexandru Săvulescu <alexandru.savulescu@epfl.ch>
Co-authored-by: Alexandru Săvulescu <alexandru.savulescu@epfl.ch>
Coverage functions 100% and lines 99.6%.
* hoc_execerr(const char* fmt, ...) more convenient for more complicated messages than hoc_execerror(const char*, const char*) Message truncated after 512 bytes. * docstring for hoc_execerr(const char fmt, ...) * Cmake configured code coverage. Enabled by '-DNRN_ENABLE_COVERAGE=ON' (default, coverage of everything). -DNRN_COVERAGE_FILES='semicolon;list;of;files;to;cover' The files are relative to PROJECT_SOURCE_DIR, e.g.src/nrniv/partrans.cpp and, for now, can be any sources underneath nrn/src. Provides: 'make cover_begin' to initialize coverage. 'make cover_html' generate an html report with all coverage since last 'make cover_begin' * Edit cmake and coverage docs * Use a KSChan HGap to help with Impedance tests. * partrans.cpp: replace OS/list.h and nrnhash.h with std:... The consequence of fewer functions increases function coverage from 79% to 93%. * partrans: -Wall warning free * partrans.cpp improve some error messages Coverage functions 100% and lines 99.6%. Co-authored-by: Pramod Kumbhar <pramod.s.kumbhar@gmail.com> Co-authored-by: Alexandru Săvulescu <alexandru.savulescu@epfl.ch>
This reverts commit 4796c3b.
* hoc_execerr(const char* fmt, ...) more convenient for more complicated messages than hoc_execerror(const char*, const char*) Message truncated after 512 bytes. * docstring for hoc_execerr(const char fmt, ...) * Cmake configured code coverage. Enabled by '-DNRN_ENABLE_COVERAGE=ON' (default, coverage of everything). -DNRN_COVERAGE_FILES='semicolon;list;of;files;to;cover' The files are relative to PROJECT_SOURCE_DIR, e.g.src/nrniv/partrans.cpp and, for now, can be any sources underneath nrn/src. Provides: 'make cover_begin' to initialize coverage. 'make cover_html' generate an html report with all coverage since last 'make cover_begin' * Edit cmake and coverage docs * Use a KSChan HGap to help with Impedance tests. * partrans.cpp: replace OS/list.h and nrnhash.h with std:... The consequence of fewer functions increases function coverage from 79% to 93%. * partrans: -Wall warning free * partrans.cpp improve some error messages Coverage functions 100% and lines 99.6%. Co-authored-by: Pramod Kumbhar <pramod.s.kumbhar@gmail.com> Co-authored-by: Alexandru Săvulescu <alexandru.savulescu@epfl.ch>
This has two new features that I find very helpful for coverage.
make cover_begin
andmake cover_html
-DNRN_COVERAGE_FILES="semicolon;separated;list;of;files;" limits coverage to those files. I've found that the workflow cycle
speeds up very satisfyingly
expect_error
function.I built with
At present in test/pynrn
gives for src/nrniv/partrans.cpp