Skip to content

Remove option to build special as a script#1289

Merged
alexsavulescu merged 2 commits into
masterfrom
remove_SCRIPT
May 20, 2021
Merged

Remove option to build special as a script#1289
alexsavulescu merged 2 commits into
masterfrom
remove_SCRIPT

Conversation

@alkino

@alkino alkino commented May 18, 2021

Copy link
Copy Markdown
Member

@codecov-commenter

codecov-commenter commented May 18, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1289 (2727559) into master (5b2da2d) will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1289      +/-   ##
==========================================
+ Coverage   32.05%   32.13%   +0.08%     
==========================================
  Files         570      570              
  Lines      108730   108730              
==========================================
+ Hits        34849    34937      +88     
+ Misses      73881    73793      -88     
Impacted Files Coverage Δ
src/parallel/bbs.cpp 64.78% <0.00%> (+1.87%) ⬆️
src/parallel/bbssrv2mpi.cpp 54.59% <0.00%> (+3.44%) ⬆️
src/nrnmpi/bbsmpipack.cpp 86.44% <0.00%> (+12.42%) ⬆️
src/parallel/bbsclimpi.cpp 51.29% <0.00%> (+21.42%) ⬆️
src/parallel/bbssrvmpi.cpp 43.03% <0.00%> (+29.11%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b2da2d...2727559. Read the comment docs.

@pramodk

pramodk commented May 18, 2021

Copy link
Copy Markdown
Member

CI is failing with errors mentioned in #1187 #1059. Wonder how -lpthread library is missing from link line.

Comment thread bin/neurondemo.in Outdated

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.

Oh...this was another can of worms.
I am not testing this locally but if CI passes then it should be good IMO.

@alexsavulescu alexsavulescu force-pushed the remove_SCRIPT branch 4 times, most recently from c6c5cd7 to 7ad08f3 Compare May 20, 2021 09:55
@alexsavulescu alexsavulescu force-pushed the remove_SCRIPT branch 7 times, most recently from 8f24027 to 02866f4 Compare May 20, 2021 13:59

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

LGTM

@alexsavulescu alexsavulescu merged commit 779b304 into master May 20, 2021
@alexsavulescu alexsavulescu deleted the remove_SCRIPT branch May 20, 2021 15:12
alkino added a commit to BlueBrain/nmodl that referenced this pull request Jan 26, 2022
alkino added a commit to BlueBrain/nmodl that referenced this pull request Jan 26, 2022
@alexsavulescu alexsavulescu mentioned this pull request Mar 22, 2022
15 tasks
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.

Make binary special as default (-DNRN_ENABLE_BINARY_SPECIAL=ON)

5 participants