Skip to content

Commit deca204

Browse files
authored
Merge pull request #28 from Remi-Gau/master
fix eyetracker folder automatic creation
2 parents 09db858 + 1bdc47f commit deca204

File tree

2 files changed

+22
-20
lines changed

2 files changed

+22
-20
lines changed

createFilename.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@
9292
[~, ~, ~] = mkdir(expParameters.outputDir);
9393
[~, ~, ~] = mkdir(fullfile(expParameters.outputDir, modality));
9494

95-
if isfield(cfg,'eyeTracker')
95+
if cfg.eyeTracker
9696
[~, ~, ~] = mkdir(fullfile(expParameters.outputDir, 'eyetracker'));
9797
end
9898

tests/test_createFilename.m

Lines changed: 21 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,13 @@ function test_createFilename()
1111
expParameters.task = 'testtask';
1212
expParameters.verbose = 1;
1313

14-
cfg.eyeTracker = true;
14+
cfg.eyeTracker = false;
1515
cfg.device = 'PC';
1616

1717
expParameters = checkCFG(cfg,expParameters);
1818
expParameters = createFilename(cfg,expParameters);
1919

20-
outputDir = fullfile(pwd, ...
20+
behDir = fullfile(pwd, ...
2121
'..', '..', ...
2222
'output', 'source', 'sub-001', 'ses-001', 'beh');
2323

@@ -26,16 +26,16 @@ function test_createFilename()
2626
'output', 'source', 'sub-001', 'ses-001', 'eyetracker');
2727

2828

29-
assert(exist(outputDir, 'dir')==7)
30-
assert(exist(eyetrackerDir, 'dir')==7)
29+
assert(exist(behDir, 'dir')==7)
30+
31+
% make sure the eyetracker dir is not created
32+
assert(exist(eyetrackerDir, 'dir')==0)
33+
3134
assert(strcmp(...
3235
expParameters.fileName.events, ...
3336
['sub-001_ses-001_task-testtask_run-001_events_date-' expParameters.date '.tsv']));
34-
assert(strcmp(...
35-
expParameters.fileName.eyetracker, ...
36-
['sub-001_ses-001_task-testtask_run-001_eyetrack_date-' expParameters.date '.edf']));
37-
3837

38+
3939
%% check directory and filename creation (fMRI)
4040

4141
clear
@@ -46,24 +46,26 @@ function test_createFilename()
4646
expParameters.runNb = 2;
4747
expParameters.task = 'testtask';
4848

49-
cfg.eyeTracker = false;
49+
cfg.eyeTracker = true;
5050
cfg.device = 'scanner';
5151

52-
expParameters = checkCFG(cfg,expParameters);
53-
expParameters = createFilename(cfg,expParameters);
54-
55-
outputDir = fullfile(pwd, ...
56-
'..', '..', ...
57-
'output', 'source', 'sub-ctrl002', 'ses-002', 'func');
52+
outputDir = fullfile(fileparts(mfilename('fullpath')), '..', 'output');
5853

59-
eyetrackerDir = fullfile(pwd, ...
60-
'..', '..', ...
61-
'output', 'source', 'sub-ctrl002', 'ses-002', 'eyetracker');
54+
funcDir = fullfile(outputDir, 'source', 'sub-ctrl002', 'ses-002', 'func');
55+
eyetrackerDir = fullfile(outputDir, 'source', 'sub-ctrl002', 'ses-002', 'eyetracker');
6256

57+
expParameters.outputDir = outputDir;
58+
expParameters = checkCFG(cfg,expParameters);
59+
expParameters = createFilename(cfg,expParameters);
6360

64-
assert(exist(outputDir, 'dir')==7)
61+
assert(exist(funcDir, 'dir')==7)
62+
assert(exist(eyetrackerDir, 'dir')==7)
6563
assert(strcmp(expParameters.fileName.base, 'sub-ctrl002_ses-002_task-testtask'))
6664
assert(strcmp(...
6765
expParameters.fileName.events, ...
6866
['sub-ctrl002_ses-002_task-testtask_run-002_events_date-' expParameters.date '.tsv']));
67+
68+
assert(strcmp(...
69+
expParameters.fileName.eyetracker, ...
70+
['sub-ctrl002_ses-002_task-testtask_run-002_eyetrack_date-' expParameters.date '.edf']));
6971

0 commit comments

Comments
 (0)