-
Notifications
You must be signed in to change notification settings - Fork 41
Update README with current execution workflow with NEURON #374
Conversation
7fcaa95
to
c9587fb
Compare
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.
Some of my edit suggestions are just a matter of taste and can be ignored.
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 README !
@pramodk took the liberty of doing a few cosmetic touches. Feel free to change/revert.
no problem, thanks!! |
Please retest |
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 took the liberty to perform my changes directly with a commit. Otherwise good to merge!
…CoreNeuron#374) * Update README with latest workflow of NEURON/CoreNEURON installation * Cleanup and integration with existing README * Add citation and contribution information * Mention flex, bison dependencies from NEURON. * Provide link to documentation for BBPCOREPOINTER fixes BlueBrain/CoreNeuron#222 BlueBrain/CoreNeuron#190 Co-authored-by: Alexandru Savulescu <46521150+alexsavulescu@users.noreply.github.com> Co-authored-by: Omar Awile <omar.awile@gmail.com> CoreNEURON Repo SHA: BlueBrain/CoreNeuron@18f35f6
Once neuronsimulator/nrn#707 will be merged, this PR shows how regular NEURON user can use CoreNEURON.
fixes #367