Skip to content
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

Controls the passing of NEURON global params to processes #165

Merged
merged 4 commits into from Apr 19, 2024
Merged

Conversation

anilbey
Copy link
Collaborator

@anilbey anilbey commented Apr 17, 2024

Do not rely on NEURON simulator for that as its behaviour varies depending on the operating system.

@anilbey anilbey marked this pull request as ready for review April 18, 2024 12:06
Copy link
Contributor

@AurelienJaquier AurelienJaquier left a comment

Choose a reason for hiding this comment

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

looks good to me

Copy link
Contributor

@ilkilic ilkilic left a comment

Choose a reason for hiding this comment

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

Thank you! Looks good to me

@anilbey anilbey merged commit ad5631b into main Apr 19, 2024
10 checks passed
@anilbey anilbey deleted the processes branch April 19, 2024 12:05
@anilbey
Copy link
Collaborator Author

anilbey commented Apr 19, 2024

An issue is created on NEURON's issue tracker with a reproducer that uses only the neuron package.
neuronsimulator/nrn#2842

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.

None yet

3 participants