Forum Replies Created
-
AuthorPosts
-
02/14/2020 at 11:21 AM in reply to: VAT problem: Ref. to non-existent field ‘vatsettings_horn_removeElectrode’ #9487tsiegerParticipant
Is the E-field “tube” expected to follow the electrode artifact?
Also, I realized there is the “Auto-segment electrode from postoperative acquisition” button in the 3D view, but it seems it does nothing for my data (postop CT).Cheers,
Tomas02/14/2020 at 11:20 AM in reply to: VAT problem: Ref. to non-existent field ‘vatsettings_horn_removeElectrode’ #9486tsiegerParticipanteven though it seems that the electrode is not that much bent:
http://neuro.felk.cvut.cz/data/uploads/images/misc/leaddbs-left-electrode.png
–tbc–02/14/2020 at 11:20 AM in reply to: VAT problem: Ref. to non-existent field ‘vatsettings_horn_removeElectrode’ #9485tsiegerParticipantI can’t even start a new topic. I’ll try to continue here:
I’m curious why the E-field deviates quite a lot from my electrode:
http://neuro.felk.cvut.cz/data/uploads/images/misc/leaddbs-vat.png–tbc–
02/14/2020 at 11:12 AM in reply to: VAT problem: Ref. to non-existent field ‘vatsettings_horn_removeElectrode’ #9480tsiegerParticipantI’m encountering problems posting replies to this thread. I’ll try to start another topic.
Tomas02/14/2020 at 11:02 AM in reply to: VAT problem: Ref. to non-existent field ‘vatsettings_horn_removeElectrode’ #9475tsiegerParticipantHi Ningfei,
thanks for your quick help.
Yes, the hotfix resolved the issue (when downloaded and applied manually). VTA then could be computed w/o problems.
BTW: installing the hotfix from the Install menu did not work: http://neuro.felk.cvut.cz/data/uploads/images/misc/leaddbs2.3-update.png- This reply was modified 4 years, 9 months ago by tsieger.
tsiegerParticipantHowever, now I run into a problem with:
Subscripted assignment dimension mismatch. Error in ea_genvat_horn>jr_remove_electrode (line 712) r_pos(:,r_pos(2,:)>0) = cr_pos; Error in ea_genvat_horn (line 355) vat = jr_remove_electrode(vat,elstruct,mesh,side,elspec); Error in ea_stimparams>stimulate_Callback (line 950) [stimparams(1,side).VAT(el).VAT,volume]=feval(ea_genvat,elstruct(el).coords_mm,getappdata(handles.stimfig,'S'),side,options,stimname,options.prefs.machine.vatsettings.horn_ethresh,handles.stimfig); Error in gui_mainfcn (line 95) feval(varargin{:}); Error in ea_stimparams (line 42) gui_mainfcn(gui_State, varargin{:}); Error in matlab.graphics.internal.figfile.FigFile/read>@(hObject,eventdata)ea_stimparams('stimulate_Callback',hObject,eventdata,guidata(hObject)) Error while evaluating UIControl Callback
size of
rpos
:
3 98658size of
r_pos(:,r_pos(2,:)>0)
:
3 89817size of
crpos
:
3 179622tsiegerParticipantAnother problem in
vecnorm
, which was introduced in R2017b.
https://github.com/netstim/leaddbs/pull/322Please note that this PR also fixed the problem with incompatible matrix sizes:
Error using - Matrix dimensions must agree. Error in ea_genvat_horn>jr_remove_electrode (line 678) tra = elstruct.trajectory{1,side} - org; Error in ea_genvat_horn (line 355) vat = jr_remove_electrode(vat,elstruct,mesh,side,elspec); Error in ea_stimparams>stimulate_Callback (line 950) [stimparams(1,side).VAT(el).VAT,volume]=feval(ea_genvat,elstruct(el).coords_mm,getappdata(handles.stimfig,'S'),side,options,stimname,options.prefs.machine.vatsettings.horn_ethresh,handles.stimfig); Error in gui_mainfcn (line 95) feval(varargin{:}); Error in ea_stimparams (line 42) gui_mainfcn(gui_State, varargin{:}); Error in matlab.graphics.internal.figfile.FigFile/read>@(hObject,eventdata)ea_stimparams('stimulate_Callback',hObject,eventdata,guidata(hObject)) Error while evaluating UIControl Callback
(I’ve used an explicit
repmat
to fix this.)tsiegerParticipantAnother problem in interactive 3d viewer control:
Error in ea_mouse_camera/motion_callback (line 63) pt = matlab.graphics.interaction.internal.getPointInPixels(hfig,currpt(1:2)); Error while evaluating Figure WindowButtonMotionFcn
this unfortunatelly also does not work on R2016a :-(.
tsiegerParticipantHi, I ran into another Matlab dependency problem with the
newline
function and create a PR fix for it: https://github.com/netstim/leaddbs/pull/321.tsiegerParticipantHi Ningfei,
I tracked the issue down. This is also a Matlab dependency problem. The problem is inea_delete.m
, which makes use of the.folder
field in the return value from thedir
function.
In the current Matlab 2019b, the entries returned contain the following fields:name folder date bytes isdir datenum
but in R2016a, they contain only:
name date bytes isdir datenum
This problem demonstrates in
ea_create_tpm_darteltemplate
, where a loop is made (l.147-194), cleaning usingea_delete
on each iteration, and crashing at the end of the first iteration, never reaching the others, such that not all templates get created.tsiegerParticipantGreat, thanks a lot.
TomastsiegerParticipantOK, VTA can be computed in native space.
However, it seems that electrode reconstruction can be performed in normalized space only, as when I attempt to pre-reconstruct electrodes without normalizing first, it fails with:glpostop_tra.nii" does not exist.
, even though I choose the ‘Native Patient Space’ option under ‘7. Visualization’.
Maybe, I’m missing something.tsiegerParticipantRe contains: I’ve issued this PR: https://github.com/netstim/leaddbs/pull/320
Re Shoot: I had no specific reason to use it, but it seems that ‘SPM12 Shoot’ is the default normalization option.
tsiegerParticipantThanks. Still, is it possible to extract electrodes in native space, in patient’s MRI being not affected in a nonlinear way?
tsiegerParticipantRegarding the
contains()
issue: I’m afraid my hack was too hacky and could not be used as a proper replacement for the original function, as it e.g. does not support patterns consisting of several strings, which the original function does support. Also,contains()
can be used in both case-sensitive and insensitive modes, whilestrfind
cannot. However, looking in the Lead-DBS code, multiple-string patterns seem not to be used, and possibly neither case-insensitive search.
So, I suppose there are three possible solutions:
i) put the hacked version ofcontains()
in the code *only* for Matlab <R12016b (I can’t see a way how to do this when there is no installation process and users can get the code directly from github),
ii) put the hacked version ofcontains()
in the code regardless of Matlab version (this would IMHO produce tons of warnings, I would not go this way),
iii) put a differently named function (e.g.mycontains()
in the code and update all the calls tocontains()
tomycontains()
– this would affect quite many people and much code, but would be quite clean IMHO.
What’s your opinion?Regarding the normalization issue: ignoring the
keyboard
did not help, the process failed with------------------------------------------------------------------------ Running job #1 ------------------------------------------------------------------------ Running 'Run Shoot (existing Templates)' Failed 'Run Shoot (existing Templates)' Error using read_hdr (line 32) Error reading header file "/home/sieger/src/matlab/leaddbs-2.2.3/templates/space/MNI_ICBM_2009b_NLIN_ASYM/dartel/shootmni_1.nii". In file "/home/sieger/src/matlab/spm12/@nifti/private/read_hdr.m" (v4967), function "read_hdr" at line 32. In file "/home/sieger/src/matlab/spm12/@nifti/nifti.m" (v4986), function "nifti" at line 26. In file "/mnt/hdd/home/sieger/src/matlab/spm12/toolbox/Shoot/spm_shoot_warp.m" (v6024), function "load_template" at line 195. In file "/mnt/hdd/home/sieger/src/matlab/spm12/toolbox/Shoot/spm_shoot_warp.m" (v6024), function "spm_shoot_warp" at line 104. The following modules did not run: Failed: Run Shoot (existing Templates) Error using MATLABbatch system Job execution failed. The full log of this run can be found in MATLAB command window, starting with the lines (look for the line showing the exact #job as displayed in this error message) ------------------ Running job #1 ------------------
The problem was that the
.../MNI_ICBM_2009b_NLIN_ASYM/dartel/shootmni_1.nii
is missing.
There are only these files in the directory:dartelmni_6.mat dartelmni_6.nii dartelmni_6_hires_00001.nii dartelmni_6_hires_00002.nii dartelmni_6_hires_00003.nii dartelmni_6_hires_00004.nii dartelmni_6_hires_00005.nii dartelmni_6_hires_00006.nii s0dartelmni_6_hires_00001.nii s0dartelmni_6_hires_00002.nii s0dartelmni_6_hires_00003.nii
-
AuthorPosts