Back to the main page.

Bug 1988 - vol.mat on ftp/tutorial/* not consistent with each other nor current code.

Status CLOSED FIXED
Reported 2013-02-15 11:03:00 +0100
Modified 2018-03-10 21:40:07 +0100
Product: FieldTrip
Component: documentation
Version: unspecified
Hardware: PC
Operating System: Windows
Importance: P3 normal
Assigned to: Lilla Magyari
URL:
Tags:
Depends on:
Blocks:
See also:

Johanna - 2013-02-15 11:03:36 +0100

ftp/tutorial/beamformer/vol.mat has a vol of type 'nolte' in 'cm' and as an example pnt: vol.bnd.pnt(500,:) 3.5939 -0.4994 1.2634 ftp/tutorial/headmodel_meg/vol.mat has a vol of type 'singleshell' in 'cm' and as an example pnt: vol.bnd.pnt(500,:) 3.7064 -0.5007 1.2089 Doing it 'manually' with today's verion, mri=ft_read_mri('/home/common/matlab/fieldtrip/data/ftp/tutorial/beamformer/Subject01.mri'); cfg = []; cfg.coordsys = 'ctf'; cfg.output = 'brain'; segmentedmri = ft_volumesegment(cfg, mri); cfg=[]; cfg.method = 'singleshell'; vol1 = ft_prepare_headmodel(cfg,segmentedmri); Gives vol1 of type 'singleshell' in 'mm' and as an example pnt: vol1.bnd.pnt(500,:) 37.0636 -5.0073 12.0887 Thus, the manual version and the headmodel_meg version match except for units. The beamformer version seems an older version. Should be updated (along with segmentedmri if that's different too?) prior to upcoming Toolkit.


Johanna - 2013-02-15 11:08:51 +0100

Also suggestion to do: Add to each test_tutorial_* function an assert-test, if the output of what is run in the function matches what is stored on ftp/tutorial/*


Johanna - 2013-02-20 13:12:23 +0100

Adding Joern and Eelke as this might affect Toolkit? I plan to fix this today.


Johanna - 2013-02-20 15:11:29 +0100

Re-assigning to Lilla: Lilla, I am blocked from updating the tutorial .mat files since you put some there which do not have group write permissions. 1) Could you please update the files, replacing the old ones, with the todays-date ones I put in the following folder? /home/common/matlab/fieldtrip/data/ftp/tutorial/headmodel_meg 2) Please change permissions of the directory headmodel_eeg, as I can not even write files to within that directory. (segmentedmri.mat needs updating). 3) See also test_bug1988.m, which now does the assert test mentioned in Comment 1


Lilla Magyari - 2013-06-12 12:33:47 +0200

(In reply to comment #3) hi Johanna, I am terrible sorry for the late reaction, I have been not seen this bug before. So far, I updated the vol and segmented mri in ftp/tutorial/headmodel_meg and I updated also the meg headmodel tutorial accordingly. I do not know how to change permissions for the heamodel_eeg, but I will look into it. Lilla


Johanna - 2013-06-12 12:49:17 +0200

chmod -R 775 headmode_eeg


Lilla Magyari - 2013-06-13 18:29:35 +0200

(In reply to comment #5) hi Johanna, all seg and vols are updated and I added two tests-cripts (one for the headmodel meg and one for the eeg tutorial) which also test if the output is the same as on the ftp site. And I also changed the permission. So, I close this bug. li