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

Sub-optimal merging of synapse groups in models with homogeneous delays #569

Open
neworderofjamie opened this issue Feb 21, 2023 · 1 comment
Milestone

Comments

@neworderofjamie
Copy link
Contributor

SynapseGroup::getWUHashDigest() is used as the basis for merging all types of synapse group but this includes the hash of both getSrcNeuronGroup()->getNumDelaySlots() and getTrgNeuronGroup()->getNumDelaySlots(). The latter is not necessary in synapse dynamics or presynaptic updates unless a postsynaptic variable is accessed but means that in models with N homogeneous delays, you end up with N^2 merged synapse groups.

@neworderofjamie
Copy link
Contributor Author

Also, all weight update model parameters are compared even if they aren't referenced e.g. in a spike-like event threshold

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant