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 functionality of SynapseView to synapse_utils #463

Open
jnsbck opened this issue Oct 23, 2024 · 0 comments
Open

Move functionality of SynapseView to synapse_utils #463

jnsbck opened this issue Oct 23, 2024 · 0 comments

Comments

@jnsbck
Copy link
Contributor

jnsbck commented Oct 23, 2024

Before #447 synapses could be "viewed" using SynapseView, i.e. using net.TestSynapse. This functionality is retained, but net.TestSynapse([0,1]) was replaced by net.TestSynapse.edge([0,1]). After discussion with @kyralianaka and @michaeldeistler we decided to remove edge, net.TestSynapse and also the channel views added in #447 (net.HH). This functionality should be moved to synapse_utils and can make heavy use net.select.

white board sketches made during the discussion
IMG_7210

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

No branches or pull requests

1 participant