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

Move threshold check outside inner loop in exponential IAF models? #2822

Closed
clinssen opened this issue Jun 5, 2023 · 1 comment
Closed
Assignees
Labels
I: Behavior changes Introduces changes that produce different results for some users S: Normal Handle this with default priority T: Discussion Still searching for the right way to proceed / suggestions welcome
Projects

Comments

@clinssen
Copy link
Contributor

clinssen commented Jun 5, 2023

The threshold check is inside the GSL integrator "inner loop" for certain exponential IAF models, e.g.:

if ( S_.y_[ State_::V_M ] < -1e3 or S_.y_[ State_::W ] < -1e6 or S_.y_[ State_::W ] > 1e6 )

is inside
while ( t < B_.step_ )

This makes sense because the membrane potential can diverge very fast, and placing the threshold check inside the inner loop makes the numerics potentially a bit more precise. However, this code arrangement is impossible to generate from NESTML.

This issue is to discuss whether having the threshold check inside the "inner loop" makes the most sense, or whether it is more consistent to have it outside that inner loop, which has the additional advantage of allowing the NESTML model to reproduce the NEST model exactly.

@clinssen clinssen added the T: Discussion Still searching for the right way to proceed / suggestions welcome label Jun 5, 2023
@terhorstd terhorstd added this to To do in Models via automation Jun 27, 2023
@terhorstd terhorstd added S: Normal Handle this with default priority I: Behavior changes Introduces changes that produce different results for some users labels Jun 27, 2023
@clinssen
Copy link
Contributor Author

clinssen commented Jul 4, 2023

This should be handled on the NESTML side; it is not an issue with NEST Simulator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: Behavior changes Introduces changes that produce different results for some users S: Normal Handle this with default priority T: Discussion Still searching for the right way to proceed / suggestions welcome
Projects
Models
  
Done
Development

No branches or pull requests

2 participants