-
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
nrnhines
left a comment
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.
alexsavulescu
left a comment
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 |
ohm314
left a comment
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 <[email protected]> Co-authored-by: Omar Awile <[email protected]> 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