Back to the main page.

Bug 3280 - electrode locations are not read from CTF *.ds res4 file

Reported 2017-04-12 20:55:00 +0200
Modified 2017-04-13 10:52:25 +0200
Product: FieldTrip
Component: fileio
Version: unspecified
Hardware: PC
Operating System: Mac OS
Importance: P5 normal
Assigned to: Robert Oostenveld
Depends on:
See also:

Robert Oostenveld - 2017-04-12 20:55:25 +0200

Marios pointed out that in an old version of the low-level reader the elec positions were available somewhere in hdr.orig, but now not any more. It will still be somewhere in the res4. The ft_read_sens (and ft_read_header and cft2grad) need some updates.

Robert Oostenveld - 2017-04-12 21:14:12 +0200

I made a branch in github for the relevant changes. mac011> git push --set-upstream origin bug3280-ctf-elec Total 0 (delta 0), reused 0 (delta 0) To * [new branch] bug3280-ctf-elec -> bug3280-ctf-elec Branch bug3280-ctf-elec set up to track remote branch bug3280-ctf-elec from origin.

Robert Oostenveld - 2017-04-12 21:25:41 +0200

Mario sent me a res4 file. I made a test script. However, I now realize that the CTF reading code requires a full dataset, not a single res4. So the test script fails prior to it getting interesting. mac011> git commit -a [bug3280-ctf-elec 85d9e9c] first test script to test the behaviour of 1 file changed, 19 insertions(+) create mode 100644 test/test_bug3280.m mac011> git push Counting objects: 4, done. Delta compression using up to 4 threads. Compressing objects: 100% (4/4), done. Writing objects: 100% (4/4), 644 bytes | 0 bytes/s, done. Total 4 (delta 2), reused 0 (delta 0) remote: Resolving deltas: 100% (2/2), completed with 2 local objects. To af02d43..85d9e9c bug3280-ctf-elec -> bug3280-ctf-elec

Robert Oostenveld - 2017-04-12 21:38:53 +0200

mac011> git commit [bug3280-ctf-elec 4396e40] make an explicit check on senstype=eeg/meg, give an error when the user's specification is inconsistent with the file content. This deals with the (undetected) error in 1 file changed, 18 insertions(+), 6 deletions(-)

Robert Oostenveld - 2017-04-13 10:49:06 +0200

I looked at it with Marios with another dataset. I made some changes and just merged it into the master (i.e. release version) of fieldtrip. See

Robert Oostenveld - 2017-04-13 10:52:25 +0200

note to myself: I still have to upload the test dataset to our central storage system.