nest / nest-simulator

The NEST simulator
http://www.nest-simulator.org
GNU General Public License v2.0
532 stars 361 forks source link

Exceptions for unused dictionary items for multiple compartments/receptors #2315

Open JoshuaBoettcher opened 2 years ago

JoshuaBoettcher commented 2 years ago

Is your feature request related to a problem? Please describe. There is not error for unused dictionary items when using SetStatus for multiple compartments and or receptors at the same time. However there is an error when adding a single compartment or a single receptor.

Describe the solution you'd like Implementation of the same DictError that occurs when single compartments are added with false parameters.

Describe alternatives you've considered N/A

Additional context For example if we want a compartmental neuron with two compartments, we currently need to add both at the same time. However the example below does not cause any errors, which makes it hard to debug the simulations since there are natually a lot of parameters in all the dictionaries.

cm = nest.Create('cm_default')

soma_params = {'unused key': 0}

dend_params = {'unused key': 0}

cm.compartments = [
    {"parent_idx": -1, "params": soma_params},
    {"parent_idx":  0, "params": dend_params}
]

receptors = [
    {"comp_idx": 0, "receptor_type": "GABA", "unused_key": 0, "params": {"unused_key": 0}},
    {"comp_idx": 0, "receptor_type": "AMPA", "unused_key": 0, "params": {"unused_key": 0}}
]

cm.receptors = receptors
github-actions[bot] commented 2 years ago

Issue automatically marked stale!

heplesser commented 1 year ago

@WillemWybo Ping!

github-actions[bot] commented 1 year ago

Issue automatically marked stale!

heplesser commented 5 months ago

Should be fixed by #2773.