-
Notifications
You must be signed in to change notification settings - Fork 9
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
Document how different entities are to be referenced #16
Comments
I'm not sure if this is consistent. When using NEURON, one must use the This works: simulation.add_column_to_output_file("output0",
column_id="pop0_0_v_Seg1_dend_1",
quantity="pop0/0/olm/7/v") but this does not: simulation.add_column_to_output_file("output0",
column_id="pop0_0_v_Seg1_dend_1",
quantity="pop0[0]/olm/7/v") It results in a NEURON writer error, so either there's a bug in the writer class, or there's another reason why this doesn't work:
I'm also unclear why one has to mention the name of the cell component ( |
Using |
@pgleeson, using indexing without the cell name does not work. So here's what I have so far. Noting this here so I can remove it from the example in the docs. simulation.add_column_to_output_file("output0",
column_id="pop0_0_v_Seg1_dend_1",
quantity="pop0/0/olm/7/v")
# TODO: why does the name of the cell component have to be included? This
# does not work, for example.
"""
simulation.add_column_to_output_file("output0",
column_id="pop0_0_v_Seg1_dend_1",
quantity="pop0/0/7/v")
"""
# TODO: why does indexing not work in this population?
"""
simulation.add_column_to_output_file("output0",
column_id="pop0_0_v_Seg1_dend_1",
quantity="pop0[0]/olm/7/v")
"""
# TODO: this does not work
"""
simulation.add_column_to_output_file("output0",
column_id="pop0_0_v_Seg1_dend_1",
quantity="pop0[0]/7/v")
""" All the cases that do not work give errors of this form:
|
Duplicate of #15 |
Most are to be referred in the
/../.../...
form following the document hierarchy. Some components where a size is provided are to be referenced using the[..]
index notation. It'll be good to document this somewhere so it is clear to users.The text was updated successfully, but these errors were encountered: