You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The variable NEST_SYNAPSE_TYPES is defined once and then used by pyNN.nest.standardmodels.synapses._get_nest_synapse_model to check for available synapse types in NEST. Before, the function called nest.Models(mtype='synapses') at every function call, which slowed down things a lot, especially in the case of many single projections.
According to Yury, in #228, NEST_SYNAPSE_TYPES needs to be re-initialised after nest.ResetKernel(), so defining it only once will not work. Perhaps move the variable into the State class?
Hi, yes that's of course an important concern.
Thus, we have to re-initialize NEST_SYNAPSE_TYPES after each nest.ResetKernel.
With "moving into the State class", do you mean setting NEST_SYNAPSE_TYPES in State.clear(), where nest.ResetKernel() is called?
However, I think we cannot circumvent the problem if the user explicitely calls nest.ResetKernel() via pyNEST.
Another point is that the list has to be updated after each new synapse model definition. Is this only done in nest/synapses.py and nest/standardmodels/synapses.py, i.e. at every creation of a connection?
jougs
changed the title
impoer NEST_SYNAPSE_TYPES from nest.__init__.py
import NEST_SYNAPSE_TYPES from nest.__init__.py
May 15, 2014
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The variable NEST_SYNAPSE_TYPES is defined once and then used by pyNN.nest.standardmodels.synapses._get_nest_synapse_model to check for available synapse types in NEST. Before, the function called nest.Models(mtype='synapses') at every function call, which slowed down things a lot, especially in the case of many single projections.