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

syn_weight semantics #23

Closed
arsenius7 opened this issue May 16, 2018 · 1 comment
Closed

syn_weight semantics #23

arsenius7 opened this issue May 16, 2018 · 1 comment

Comments

@arsenius7
Copy link
Contributor

According to the current spec,

syn_weight - Strength of the connection between the source and target nodes. The units depend on the requirements of the target mechanism. For example, if the target mechanism is NEURON's Exp2Syn, then the syn_weight is interpreted as a peak conductance measured in microSiemens (uS)

Imho, reusing same property name with different semantics can easily lead to misintepretations (at least, from less simulation-savvy people, but the format should be defensive against this type of users).

It should be either given different names depending on the model used (say, conductance for Exp2Syn vs syn_weight for instanteneous), or at least have an attribute with corresponding units (uS vs ?); ideally both.

@arsenius7
Copy link
Contributor Author

Having units would have greatly reduced the chance for misinterpretation.
Closing the issue in favour of #66.

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

2 participants