-
Notifications
You must be signed in to change notification settings - Fork 116
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
allow for two point (single section) SWC somas #1144
Conversation
* add test for simple two point swc soma * only test for neuromorpho three point soma if the SWC file has a soma of three points
soma3geom = 0 | ||
if (nsoma_pts == 3 && pix2ix(1) == 0 && pix2ix(2) == 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
previously, for 2 points, this would cause an error:
NEURON: subscript out of range x
near line 0
}
^
Import3d_SWC_read[13].pix2ix(2)
Import3d_SWC_read[13].check_soma3geom(2, Vector[1450])
Import3d_SWC_read[13].sectionify()
Import3d_SWC_read[13].input("temp.tmp")
My hoc isn't great, but I take it that it doesn't do short-circuit evaluation?
The original SWC format paper (An on-line archive of reconstructed hippocampal neurons) says this: So, I believe that a single cylinder is an allowable value for the soma. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Too bad about the lack of short circuit evaluation in an if statement.
thanks @mgeplf! |
due to neuronsimulator/nrn#1144 This reverts commit 24d7c1d
due to neuronsimulator/nrn#1144 This reverts commit 24d7c1d
* add test for simple two point swc soma * only test for neuromorpho three point soma if the SWC file has a soma of three points
* add test for simple two point swc soma * only test for neuromorpho three point soma if the SWC file has a soma of three points
a soma of three points