Merge pull request #24 from cpp-lln-lab/bump_bids_matlab

[ENH] Bump bids matlab

Author: Remi-Gau

.github/workflows/miss_hit_quality.yml

@@ -30,8 +30,6 @@ jobs:
       run: |
         python -m pip install --upgrade pip setuptools
         pip3 install -r requirements.txt
-        cd tests
-        make data
 
     - name: MISS_HIT Metrics
       run: |

.github/workflows/miss_hit_style.yml

@@ -7,7 +7,7 @@ on:
       - master
       - dev
   pull_request:
-    branches: '*'
+    branches: ['*']
 
 jobs:
   build:
@@ -30,8 +30,6 @@ jobs:
       run: |
         python -m pip install --upgrade pip setuptools
         pip3 install -r requirements.txt
-        cd tests
-        make data
 
     - name: MISS_HIT Code style
       run: |

src/roi/get_ROI_Coordinates.m renamed to demos/WIP/get_ROI_Coordinates.m

@@ -1,9 +1,14 @@
 function mni = get_ROI_Coordinates
   % This function gets the individual subject coordniates of the highest peak
-  % within a specified Region of interest (usually coming from the group level univariate analysis)
-
+  % within a specified Region of interest (usually coming
+  % from the group level univariate analysis)
+  %
   % Critical t-value for each experimental condition or mask file
-  CriticalTs = 1;               % Critical t-value for visual condition in L-V5 and R-V5 and bilateral PT
+  % Critical t-value for visual condition in L-V5 and R-V5 and bilateral PT
+  %
+  % (C) Copyright 2021 CPP ROI developers
+
+  CriticalTs = 1;
 
   %% load the data structure
   WD = pwd;
@@ -52,9 +57,17 @@
         %% the first 4 masks are for the FACE condition, the other 4
         % are from the SCENE condition
         if iMask <= 2
-          result_file = [data_path, '/', SubName, '/stats/ffx_visMotion/ffx_', smoothing, '/spmT_0013.nii']; % HUMAN > BIG_ENV
+          result_file = [data_path, '/', ...
+                         SubName, ...
+                         '/stats/ffx_visMotion/ffx_', ...
+                         smoothing, ...
+                         '/spmT_0013.nii']; % HUMAN > BIG_ENV
         else
-          result_file = [data_path, '/', SubName, '/stats/ffx_audMotion/ffx_', smoothing, '/spmT_0014.nii']; % BIG_ENV > HUMAN
+          result_file = [data_path, '/', ...
+                         SubName, ...
+                         '/stats/ffx_audMotion/ffx_', ...
+                         smoothing, ...
+                         '/spmT_0014.nii']; % BIG_ENV > HUMAN
         end
 
         %%
@@ -78,8 +91,11 @@
 
           % convert space from slice number to mni
           mni{1, iMask}(subCounter, :) = cor2mni([x y z], mask_path);
-          %               mni{1,iMask}(iSub,1) = mni{1,iMask}(iSub,1)* -1;   % If  masks created from AFNI or FSL,
-          %               the x coordinate could be flipped (multiplied x -1). If this is the case, multiply x with -1.
+
+          %               mni{1,iMask}(iSub,1) = mni{1,iMask}(iSub,1)* -1;
+          % If  masks created from AFNI or FSL,
+          % the x coordinate could be flipped (multiplied x -1).
+          % If this is the case, multiply x with -1.
 
           mni{1, iMask}(subCounter, :);
 

docs/CONTRIBUTING.md

@@ -18,7 +18,3 @@ help:
 # "make mode" option.  $(O) is meant as a shortcut for $(SPHINXOPTS).
 %: Makefile
 	@$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
-
-
-doc:
-	@$(SPHINXBUILD) -b html "$(SOURCEDIR)" "$(BUILDDIR)"

docs/Makefile

@@ -23,18 +23,15 @@
 author = "the CPP ROI dev team"
 
 # The full version, including alpha/beta/rc tags
-release = "v0.1.0"
+release = "v0.1.0dev"
 
 
 # -- General configuration ---------------------------------------------------
 
 # Add any Sphinx extension module names here, as strings. They can be
 # extensions coming with Sphinx (named 'sphinx.ext.*') or your custom
 # ones.
-extensions = [
-    'sphinxcontrib.matlab',
-    'sphinx.ext.autodoc',
-    'sphinx_copybutton']
+extensions = ["sphinxcontrib.matlab", "sphinx.ext.autodoc", "sphinx_copybutton"]
 matlab_src_dir = os.path.dirname(os.path.abspath("../../src"))
 primary_domain = "mat"
 

docs/source/conf.py

@@ -6,7 +6,7 @@
 
 spm('defaults', 'fMRI');
 
-testFolder = fullfile(fileparts(mfilename('fullpath')));
+testFolder = fullfile(fileparts(mfilename('fullpath')), 'tests');
 
 folderToCover = fullfile(testFolder, '..', 'src');
 

tests/run_tests.m renamed to run_tests.m

@@ -23,7 +23,7 @@
   outputVol(vol == labelStruct.label) = true;
 
   p = bids.internal.parse_filename(sourceImage);
-  p.entities.label = labelStruct.ROI;
+  p.entities.label = bids.internal.camel_case(labelStruct.ROI);
   p.suffix = 'mask';
 
   bidsFile = bids.File(p);

src/atlas/extractRoiByLabel.m

@@ -29,7 +29,7 @@
 
   basename = spm_file(inputImage, 'basename');
   parts = strsplit(basename, '_');
-  p.entities.label = ['neurosynth ' parts{1}];
+  p.entities.label = bids.internal.camel_case(['neurosynth ' parts{1}]);
 
   bidsFile = bids.File(p);
 

src/roi/renameNeuroSynth.m

@@ -45,7 +45,7 @@
   p = bids.internal.parse_filename(inputImage);
 
   p.suffix = 'mask';
-  
+
   % add peakThreshold and clusterSizeInfo to desc
   if ~isfield(p.entities, 'desc')
     p.entities.desc = '';