-
Notifications
You must be signed in to change notification settings - Fork 45
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
Add CoCo to check if the random number functions in NEST #1104
Conversation
Would you say this PR fixes #804 and it can be closed? |
tests/nest_tests/resources/random_functions_illegal_neuron.nestml
Outdated
Show resolved
Hide resolved
Thanks, this looks great other than some very small comments! |
fix code generator options handling
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.
Many thanks, just a tiny comment, please feel free to merge this PR afterwards!
Bencher Report
Click to view all benchmark results
|
This CoCo checks if the functions associated with random numbers are used only in the
update
block. This CoCo applies only to the NEST simulator as using them instate
,parameters
, andinternals
blocks can lead to unexpected results.Fixes #804