Merge pull request #787 from Remi-Gau/validate

[ENH] improve bids validation

Author: Remi-Gau

.github/workflows/run_system_tests_matlab.yml

@@ -64,12 +64,23 @@ jobs:
           # MATLAB release to set up R2020a
           release: R2020a
 
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 18
+
+      - uses: actions/setup-python@v4
+        with:
+          python-version: '3.10'
+
       - name: Clone bidspm
         uses: actions/checkout@v3
         with:
           submodules: recursive
           fetch-depth: 0
 
+      - name: Install validators
+        run: make install
+
       - name: Get moae fmriprep data from OSF
         run: |
           mkdir -p demos/MoAE/inputs/

.github/workflows/run_system_tests_octave.yml

@@ -59,6 +59,14 @@ jobs:
         sudo apt-get -y -qq update
         sudo apt-get -y install octave liboctave-dev
 
+    - uses: actions/setup-node@v3
+      with:
+        node-version: 18
+
+    - uses: actions/setup-python@v4
+      with:
+        python-version: '3.10'
+
     - name: Info
       run: |
         octave --version
@@ -69,6 +77,9 @@ jobs:
         submodules: recursive
         fetch-depth: 0
 
+    - name: Install validators
+      run: make install
+
     - name: Get moae fmriprep data from OSF
       run: |
         mkdir -p demos/MoAE/inputs/

.github/workflows/run_tests_matlab.yml

@@ -29,12 +29,23 @@ jobs:
           # MATLAB release to set up R2020a
           release: R2020a
 
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 18
+
+      - uses: actions/setup-python@v4
+        with:
+          python-version: '3.10'
+
       - name: Clone bidspm
         uses: actions/checkout@v3
         with:
           submodules: recursive
           fetch-depth: 0
 
+      - name: Install validators
+        run: make install
+
       - name: Install SPM
         run: |
           git clone https://github.com/spm/spm12.git --depth 1

.github/workflows/run_tests_octave.yml

@@ -39,12 +39,23 @@ jobs:
       run: |
         octave --version
 
+    - uses: actions/setup-node@v3
+      with:
+        node-version: 18
+
+    - uses: actions/setup-python@v4
+      with:
+        python-version: '3.10'
+
     - name: Clone bidspm
       uses: actions/checkout@v3
       with:
         submodules: recursive
         fetch-depth: 0
 
+    - name: Install validators
+      run: make install
+
     - name: Install SPM
       run: |
         git clone https://github.com/spm/spm12.git --depth 1

.gitmodules

@@ -31,6 +31,3 @@
 	path = lib/octache
 	url = https://github.com/Remi-Gau/Octache.git
 	datalad-url = https://github.com/Remi-Gau/Octache.git
-[submodule "tools/stats_model"]
-	path = tools/stats_model
-	url = https://github.com/bids-standard/stats-models.git

.pre-commit-config.yaml

@@ -53,8 +53,20 @@ repos:
   rev: 22.10.0
   hooks:
   - id: black
+    args: [--config=pyproject.toml]
 
 - repo: https://github.com/asottile/reorder_python_imports
   rev: v3.9.0
   hooks:
   - id: reorder-python-imports
+
+- repo: https://github.com/asottile/setup-cfg-fmt
+  rev: v2.1.0
+  hooks:
+  - id: setup-cfg-fmt
+
+- repo: https://github.com/asottile/pyupgrade
+  rev: v3.1.0
+  hooks:
+  - id: pyupgrade
+    args: [--py38-plus]

.readthedocs.yml

@@ -21,6 +21,6 @@ formats:
 
 # Optionally set the version of Python and requirements required to build your docs
 python:
-  version: 3.8
+  version: "3.8"
   install:
     - requirements: requirements_doc.txt

CITATION.cff

@@ -2,7 +2,7 @@ cff-version: 1.2.0
 
 title: "bidspm"
 
-version: 2.1.0dev
+version: 2.2.0
 
 abstract: BIDSpm is a set pipelines and tools for Octave/MATLAB to process and analyze BIDS data sets using SPM.
 

Dockerfile

@@ -1,4 +1,6 @@
-# Creates a docker image of bidspm with
+# Creates a docker image of bidspm
+
+# version number are updated automatically with tbe bump version script
 
 # this is mostly taken from the spm docker files: https://github.com/spm/spm-docker
 FROM ubuntu:22.04
@@ -8,62 +10,59 @@ USER root
 ENV LANG="en_US.UTF-8" \
     LC_ALL="en_US.UTF-8"
 
-#TODO how to update this with the content of the current version
-LABEL version="1.1.5dev"
+LABEL version="2.2.0"
 
 LABEL maintainer="Rémi Gau <remi.gau@gmail.com>"
 
-## Install SPM
-# basic OS tools install and also octave
+## basic OS tools install, node, npm also octave
 RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get -y install \
-    build-essential \
+    build-essential software-properties-common\
     curl \
-    octave \
-    liboctave-dev &&
-    apt-get clean &&
+    octave liboctave-dev \
+    nodejs npm
+
+## add python
+RUN add-apt-repository ppa:deadsnakes/ppa && apt-get update
+RUN apt-get -y install python3.10 python3-pip
+
+RUN apt-get clean && \
     rm -rf \
         /tmp/hsperfdata* \
         /var/*/apt/*/partial \
         /var/lib/apt/lists/* \
         /var/log/apt/term*
 
-RUN mkdir /opt/spm12 &&
-    curl -SL https://github.com/spm/spm12/archive/r7771.tar.gz |
-    tar -xzC /opt/spm12 --strip-components 1 &&
-    curl -SL https://raw.githubusercontent.com/spm/spm-docker/main/octave/spm12_r7771.patch |
-    patch -p0 &&
-    make -C /opt/spm12/src PLATFORM=octave distclean &&
-    make -C /opt/spm12/src PLATFORM=octave &&
-    make -C /opt/spm12/src PLATFORM=octave install &&
+## Install SPM
+RUN mkdir /opt/spm12 && \
+    curl -SL https://github.com/spm/spm12/archive/r7771.tar.gz | \
+    tar -xzC /opt/spm12 --strip-components 1 && \
+    curl -SL https://raw.githubusercontent.com/spm/spm-docker/main/octave/spm12_r7771.patch | \
+    patch -p0 && \
+    make -C /opt/spm12/src PLATFORM=octave distclean && \
+    make -C /opt/spm12/src PLATFORM=octave && \
+    make -C /opt/spm12/src PLATFORM=octave install && \
     ln -s /opt/spm12/bin/spm12-octave /usr/local/bin/spm12
-
 RUN octave --no-gui --eval "addpath('/opt/spm12/'); savepath ();"
 
 ## Install nods and bids validator
-RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get -y install \
-    nodejs npm &&
-    apt-get clean &&
-    rm -rf \
-        /tmp/hsperfdata* \
-        /var/*/apt/*/partial \
-        /var/lib/apt/lists/* \
-        /var/log/apt/term*
-
 RUN node -v && npm -v && npm install -g bids-validator
 
 ## Install BIDSpm in user folder
 RUN test "$(getent passwd neuro)" || useradd --no-user-group --create-home --shell /bin/bash neuro
 
-USER neuro
-
 WORKDIR /home/neuro
+RUN mkdir code input output
 
-RUN mkdir code output bidspm
+# COPY . /home/neuro/bidspm # this is for local development
+RUN git clone --branch v2.2.0 --depth 1 --recursive https://github.com/cpp-lln-lab/bidspm.git
 
-WORKDIR /home/neuro/
+RUN cd bidspm && pip3 install .
+RUN echo '\n'
+RUN python3 --version && pip3 list
+RUN echo '\n'
 
-COPY [".", "/home/neuro/bidspm/"]
+RUN cd bidspm && octave --no-gui --eval "bidspm; savepath();"
 
-RUN cd bidspm && octave --no-gui --eval "initCppSpm(false); savepath();"
+USER neuro
 
 ENTRYPOINT ["octave"]