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

Can't specify synapse label when using array indices and one-to-one connection rule #2285

Closed
zbarni opened this issue Feb 2, 2022 · 1 comment · Fixed by #2298
Closed
Assignees
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: High Should be handled next stale Automatic marker for inactivity, please have another look here T: Bug Wrong statements in the code or documentation
Projects

Comments

@zbarni
Copy link

zbarni commented Feb 2, 2022

Describe the bug

It's not possible to use pass a synapse_label parameter in the syn_spec dictionary, in conjunction with the one_to_one rule and node ids passed as arrays. It works when nodes are passed as NodeCollection, but this limits some use cases where the same node should appear multiple times.

To Reproduce

import nest
neurons = nest.Create('iaf_psc_exp', 2)
nest.Connect([1], [2], conn_spec='one_to_one', syn_spec={'synapse_model': 'static_synapse_lbl', 'synapse_label': 1})

gives the error

>> nest.lib.hl_api_exceptions.TypeMismatch: TypeMismatch in SLI function connect_arrays

However, the following works:

neurons = nest.Create('iaf_psc_exp', 2)
nest.Connect(nest.NodeCollection([1]), nest.NodeCollection([2]),
             conn_spec='one_to_one', syn_spec={'synapse_model': 'static_synapse_lbl', 'synapse_label': 1})

Removing the synapse_label parameter also works.

Expected behavior

Connection should be successful.

Desktop/Environment (please complete the following information):
cmake_summary.txt

  • OS: Linux Mint 19.3, kernel 5.4.0-050400-generic
  • Python-Version: 3.9.0
  • NEST-Version: 3.2
  • Installation: manual install inside Miniconda env, see attached file.
@stinebuu stinebuu added I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: High Should be handled next T: Bug Wrong statements in the code or documentation labels Feb 8, 2022
@stinebuu stinebuu added this to To do (open issues) in Kernel via automation Feb 8, 2022
@github-actions
Copy link

Issue automatically marked stale!

@github-actions github-actions bot added the stale Automatic marker for inactivity, please have another look here label Apr 10, 2022
Kernel automation moved this from To do (open issues) to Done (PRs and issues) Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: High Should be handled next stale Automatic marker for inactivity, please have another look here T: Bug Wrong statements in the code or documentation
Projects
Kernel
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants