comment if / else block in setSpeedTargets and add a TODO

Author: Remi-Gau

subfun/design/setSpeedTargets.m

@@ -1,24 +1,25 @@
 function speeds = setSpeedTargets(cfg)
-    % If selected as task, it omputes the matrix with the speeds set in
+    %
+    % USAGE::
+    %
+    %  speeds = setSpeedTargets(cfg)
+    %
+    % If selected as task, it computes the matrix with the speeds set in
     % ``cfg.dot.speed`` (computed in pixels per frame) and the target ones (higher = faster,
     % lower = slower).
     %
-    % If this task is not requeste, the output will be a matrix where the values correspond to the
+    % If this task is not requested, the output will be a matrix where the values correspond to the
     % set speed in ``cfg.dot.speed`` (computed in pixels per frame).
     %
-    % This function is in * * W I P * *, the output is the same if the task is selected or not.
     %
     % (C) Copyright 2021 CPP visual motion localizer developpers
 
-    if ismember('speed', cfg.target.type)
-        [~, nbEventsPerBlock, ~, nbBlocks] = getDesignInput(cfg);
-        speeds = ones(nbBlocks, nbEventsPerBlock) * cfg.dot.speedPixPerFrame;
-
-        % Outputs an "empty" matrix in case no speed task is required
-    else
-        [~, nbEventsPerBlock, ~, nbBlocks] = getDesignInput(cfg);
-        speeds = ones(nbBlocks, nbEventsPerBlock);
+    [~, nbEventsPerBlock, ~, nbBlocks] = getDesignInput(cfg);
+    speeds = ones(nbBlocks, nbEventsPerBlock) * cfg.dot.speedPixPerFrame;
 
-    end
+    % TODO: make the output different whether the task is selected or not.
+    %     if ismember('speed', cfg.target.type)
+    %     else % Outputs an "empty" matrix in case no speed task is required
+    %     end
 
 end

tests/test_setSpeedTargets.m

@@ -20,7 +20,6 @@ function test_setSpeedTargetsBasic()
 
     speeds = setSpeedTargets(cfg);
 
-    % not sure why the fixation cross should have a speed ????
-    assertEqual(speeds, ones(24, 12));
+    assertEqual(speeds, ones(24, 12) * 28);
 
 end