Fixed matrix size on subject assertion. Wrong index

Author: fabriciocravo

atlas_storage/test_hcp_fc_atlas.mat

@@ -72,15 +72,6 @@
             RP.n_subs_2 = n_subs_2;
             RP.n_subs = n_subs_1 + n_subs_2;
 
-            n_available = size(BrainData.(ref_cond).data,1);
-
-            assert(RP.n_subs <= n_available, ...
-                ['Dataset bug: t2 groups sum to %d subjects but BrainData.%s ' ...
-                'only has %d unique subjects. ' ...
-                'Groups likely overlap (a subject matched both conditions) ' ...
-                'or IDs are duplicated.'], ...
-                RP.n_subs, ref_cond, n_available);
-
         case 'r'
 
             ref_cond = TestData.reference_condition;
@@ -117,7 +108,16 @@
             RP.n_subs_2 = length(sub_ids);
             RP.n_subs = length(sub_ids);
 
+
     end 
 
+    n_available = size(BrainData.(ref_cond).data,2);
+
+    assert(RP.n_subs <= n_available, ...
+        ['Dataset bug: t2 groups sum to %d subjects but BrainData.%s ' ...
+        'only has %d unique subjects. ' ...
+        'Groups likely overlap (a subject matched both conditions) ' ...
+        'or IDs are duplicated.'], ...
+        RP.n_subs, ref_cond, n_available);
 
 end

data/test_act_act.mat

@@ -49,14 +49,14 @@
 Params.recalculate = false;
 
 %%% Resampling parameters %%%
-Params.parallel = false; % run stuff sequentially or in parallel
-Params.n_workers = 10; % num parallel workers for parfor, best if # workers = # cores
+Params.parallel = true; % run stuff sequentially or in parallel
+Params.n_workers = 5; % num parallel workers for parfor, best if # workers = # cores
 Params.n_repetitions = 100;  % 500 recommended
 Params.batch_size = 10;
 
 %% Skip some tests - change ranges or the function
 % Tests to skip accepts functions
-ranges = {[0, 1]};
+ranges = {[0, 0]};
 Params.tests_to_skip = @(x) any(cellfun(@(r) (x >= r(1)) && (x <= r(2)), ranges));
 
 %% List of subjects per subset
@@ -80,7 +80,7 @@
 %%%%% DEVELOPERS ONLY %%%%%
 % Use a small subset of permutations for faster development -- inappropriate for inference
 
-Params.testing = true;
+Params.testing = false;
 Params.test_n_perms = 5;
 Params.test_n_repetitions = 10;
 Params.test_n_workers = 1;