Back to the main page.

Bug 2741 - fix cifti indexing issue with missing data on medial wall

Status CLOSED FIXED
Reported 2014-10-17 14:27:00 +0200
Modified 2015-02-11 10:40:27 +0100
Product: FieldTrip
Component: fileio
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P5 normal
Assigned to: Robert Oostenveld
URL:
Tags:
Depends on:
Blocks:
See also:

Robert Oostenveld - 2014-10-17 14:27:09 +0200

this should address the indexing issue reported as http://bugzilla.fcdonders.nl/show_bug.cgi?id=2096#c22


Robert Oostenveld - 2014-10-17 14:52:06 +0200

I found the problem. Again thanks for reporting. Thit should fix it: mac011> svn commit fileio/ft_read_cifti.m test/test_bug2741.m Sending fileio/ft_read_cifti.m Adding test/test_bug2741.m Transmitting file data .. Committed revision 9910. In short: I copied the test data to /home/common/matlab/fieldtrip/data/test/bug2741 and wrote the following test script ---- cii1 = ft_read_cifti('zstat1.dtseries.nii', 'cortexleft', '100307.L.midthickness.32k_fs_LR.surf.gii', 'cortexright', '100307.R.midthickness.32k_fs_LR.surf.gii'); ft_write_cifti('zstat2', cii1, 'parameter', 'dtseries'); cii2 = ft_read_cifti('zstat2.dtseries.nii'); % surfaces will be read automatically cii1 = rmfield(cii1, 'hdr'); cii2 = rmfield(cii2, 'hdr'); assert(isequaln(cii1, cii2)); ---- which now works.


Robert Oostenveld - 2014-10-18 17:51:29 +0200

although it now works for the file you provided, with test_bug2096 I noticed that it now fails for a set of cifti-1 test files that were previously working fine.


Robert Oostenveld - 2014-10-18 18:06:51 +0200

(In reply to Robert Oostenveld from comment #2) fixed, now ok with test_bug2096 and test_bug2741. mac011> svn commit fileio/ft_read_cifti.m Sending fileio/ft_read_cifti.m Transmitting file data . Committed revision 9915.


Robert Oostenveld - 2015-02-11 10:40:27 +0100

Closed several bugs that were recently resolved. Please reopen if you are not happy with the resolution.