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