Back to the main page.
Bug 3280 - electrode locations are not read from CTF *.ds res4 file
| Status | REOPENED |
| 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 |
| URL: | |
| Tags: | |
| Depends on: | |
| Blocks: | |
| 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 github.com:robertoostenveld/fieldtrip.git * [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 http://bugzilla.fieldtriptoolbox.org/show_bug.cgi?id=3280 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 github.com:robertoostenveld/fieldtrip.git 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 http://bugzilla.fieldtriptoolbox.org/show_bug.cgi?id=3280 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 https://github.com/fieldtrip/fieldtrip/pull/405