Skip to content
This repository has been archived by the owner on Mar 20, 2023. It is now read-only.

Add support for inserting nmodl flags in nrnivmodl-core calls #398

Merged
merged 3 commits into from
Oct 9, 2020

Conversation

cattabiani
Copy link
Contributor

@cattabiani cattabiani commented Oct 7, 2020

For example, this enables the call:

nrnivmodl-core -a "sympy --analytic" mod/

Partially addresses issue #344

For example, this enables the call:

nrnivmodl-core -a "sympy --analytic" mod/
@bbpbuildbot
Copy link
Collaborator

Can one of the admins verify this patch?

1 similar comment
@bbpbuildbot
Copy link
Collaborator

Can one of the admins verify this patch?

@cattabiani cattabiani self-assigned this Oct 7, 2020
@cattabiani cattabiani closed this Oct 7, 2020
@cattabiani cattabiani reopened this Oct 7, 2020
@cattabiani cattabiani requested a review from pramodk October 9, 2020 07:43
Implementing the logic to sanitize the merge of default and
runtime nmodl flags was a little too much for a makefile.
We let the user add the runtime option at his own risk. We
inform him on what were the default flags to let him make an
easy merge with what he wants.

- Add warning and info to be explicit
@cattabiani
Copy link
Contributor Author

please retest

@cattabiani cattabiani marked this pull request as ready for review October 9, 2020 15:14
Copy link
Collaborator

@pramodk pramodk left a comment

Choose a reason for hiding this comment

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

LGTM

@pramodk pramodk merged commit a3797b9 into master Oct 9, 2020
@pramodk pramodk deleted the katta/nmodl_flags_runtime branch October 9, 2020 15:26
pramodk pushed a commit to neuronsimulator/nrn that referenced this pull request Nov 2, 2022
…ain/CoreNeuron#398)

* Add support for inserting nmodl flags in nrnivmodl-core calls
    For example, this enables the call:
        nrnivmodl-core -a "sympy --analytic" mod/
* Runtime nmodl flags replace build nmodl flags

CoreNEURON Repo SHA: BlueBrain/CoreNeuron@a3797b9
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants