Skip to content

Commit

Permalink
Added a warning when spike-like-event times are required but there ar…
Browse files Browse the repository at this point in the history
…e multiple conditions
  • Loading branch information
neworderofjamie committed Dec 16, 2020
1 parent 9a088d7 commit 5ffb045
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/genn/genn/modelSpec.cc
Original file line number Diff line number Diff line change
Expand Up @@ -200,6 +200,9 @@ void ModelSpec::finalize()
}
}
if (n.second.getSpikeEventCondition().size() > 1) {
if(n.second.isSpikeEventTimeRequired() || n.second.isPrevSpikeEventTimeRequired()) {
LOGW << "Neuron group '" << n.first << "' records spike-like-event times but, it has outgoing synapse groups with multiple spike-like-event conditions so the recorded times may be ambiguous.";
}
for(auto *sg : n.second.getOutSyn()) {
if (!sg->getWUModel()->getEventCode().empty()) {
sg->setEventThresholdReTestRequired(true);
Expand Down

0 comments on commit 5ffb045

Please sign in to comment.