Closed aarongeller closed 1 year ago
Hi Aaron, sorry this took a while to merge but your fix has been addressed in #3. If this does not work for you or if you notice some other bug, feel free to open another issue/pull request. Thanks for bringing this to our attention!
no worries, thanks for the update and happy to help!
Hi @aarongeller, did you change anything else in OPSCEA.m regarding this issue? specifically, an issue has surfaced with our data where the value in depths{i}(end)
is bigger than what is allowed by the size of em
(e.g. there is a call like this em(324,1)
when em only has ~150 rows. Did you address this possibility? Otherwise I will look at it on my end. Thanks a lot!
Nevermind, I think our params file was the problem. Sorry to bother you
no worries- I was going to say there are no other modifications (for this issue) beyond those I mentioned above.
There appears to be a minor bug which prevents OPSCEA from working (fully) on bilateral implants. Specifically, the isL and isR variables are defined as scalars at OPSCEA.m (line 154) but need to be N-dimensional logical vectors where N is the number of depths. In order for OPSCEAsurfslice to adjust the view angle correctly for the side of the depth electrode (see e.g. the attached image with slices for left contacts angled to the patient's right and vice versa), the side should be specified for each.
I have modified OPSCEA.m with:
And OPSCEAsurfslice.m with:
This issue also causes left sided slices to be rendered backwards, as the line
theta=theta+[pi*loaf.isL]
does not work correctly.