Skip to content

Commit

Permalink
Merge remote-tracking branch 'upstream/main' into macs_gof
Browse files Browse the repository at this point in the history
  • Loading branch information
Remi-Gau committed Sep 20, 2023
2 parents 696c8e7 + 3a51c8b commit 3cb26c0
Show file tree
Hide file tree
Showing 15 changed files with 423 additions and 394 deletions.
2 changes: 1 addition & 1 deletion .dockerignore
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ coverage_html
templates
**/tests
**/bids-examples
lib/*/.git
lib/**/.git


# General
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/tests_octave.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ jobs:
- name: Install python
uses: actions/setup-python@v4
with:
python-version: '3.10'
python-version: '3.11'

- name: Clone bidspm
uses: actions/checkout@v3
Expand Down
4 changes: 2 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ repos:
args: [--config=pyproject.toml]

- repo: https://github.com/asottile/pyupgrade
rev: v3.10.1
rev: v3.11.0
hooks:
- id: pyupgrade
args: [--py38-plus]
Expand All @@ -73,7 +73,7 @@ repos:
- id: flynt

- repo: https://github.com/asottile/reorder-python-imports
rev: v3.10.0
rev: v3.11.0
hooks:
- id: reorder-python-imports
args: [--py38-plus, --add-import, from __future__ import annotations]
Expand Down
6 changes: 4 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ ARG = -nodisplay -nosplash -nodesktop

install:
npm install -g bids-validator
pip3 install .
pip3 install .[dev]

help: ## Show what this Makefile can do
@python -c "$$PRINT_HELP_PYSCRIPT" < $(MAKEFILE_LIST)
Expand Down Expand Up @@ -122,7 +122,6 @@ clean_docker:

build_image: Dockerfile ## Build stable docker image from the main branch
docker build . -f Dockerfile -t cpplab/bidspm:unstable
VERSION=$(cat version.txt | cut -c2-)

Dockerfile_matlab:
docker run --rm kaczmarj/neurodocker:0.9.1 generate docker \
Expand All @@ -138,4 +137,7 @@ Dockerfile_matlab:
build_image_matlab: Dockerfile_matlab
docker build . -f Dockerfile_matlab -t cpplab/bidspm_matlab:unstable

docker_data:
make -C demos/openneuro data_ds000001

################################################################################
13 changes: 11 additions & 2 deletions demos/MoAE/test_moae.m
Original file line number Diff line number Diff line change
Expand Up @@ -84,13 +84,22 @@

cd(WD);

% with Octave running more n-1 loop in CI is fine
% Octave
% running more n-1 loop in CI is fine
% but not running crashes with a segmentation fault
% /home/runner/work/_temp/fb8e9d58-fa9f-4f93-8c96-387973f3632e.sh: line 2:
% 7487 Segmentation fault (core dumped) octave $OCTFLAGS --eval "run system_tests_facerep;"
%
% not sure why
if bids.internal.is_octave()
%

% Windows
% Error using overwriteDir
% C:\Users\runneradmin\AppData\Local\Temp\8f7c44ca98d0\outputs\derivatives\bidspm-stats\
% sub-01\task-auditory_space-individual_FWHM-6 could not be removed.
% Error in setBatchSubjectLevelGLMSpec (line 73)
% overwriteDir(ffxDir, opt);
if bids.internal.is_octave() || ispc()
break
end

Expand Down
1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ dev = [
"coverage",
"pytest",
"ruamel.yaml",
'pandas'
]

[project.scripts]
Expand Down
2 changes: 1 addition & 1 deletion tests/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ clean_local:
rm -rf data/derivatives/bidspm-*/jobs

create_dummy_dataset: clean_local
sh createDummyDataSet.sh
python create_dummy_dataset.py

bids_examples:
rm -rf bids-examples/
Expand Down
Loading

0 comments on commit 3cb26c0

Please sign in to comment.