Skip to content

Commit

Permalink
Fix bids-validator installation and usage (#76)
Browse files Browse the repository at this point in the history
  • Loading branch information
tsalo authored Sep 23, 2024
1 parent 2f1aade commit df73880
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ RUN micromamba create -y -f /tmp/env.yml && \
# Check if this is still necessary when updating the base image.
ENV PATH="/opt/conda/envs/fmripost_aroma/bin:$PATH" \
UV_USE_IO_URING=0
RUN npm install -g svgo@^3.2.0 bids-validator@^1.14.0 && \
RUN npm install -g svgo@^3.2.0 [email protected].10 && \
rm -r ~/.npm

#
Expand Down
5 changes: 3 additions & 2 deletions src/fmripost_aroma/cli/parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -540,6 +540,7 @@ def parse_args(args=None, namespace=None):

bids_dir = config.execution.bids_dir
output_dir = config.execution.output_dir
derivatives = config.execution.derivatives
work_dir = config.execution.work_dir
version = config.environment.version

Expand Down Expand Up @@ -571,8 +572,8 @@ def parse_args(args=None, namespace=None):
'Please modify the output path.'
)

# Validate inputs
if not opts.skip_bids_validation:
# Validate raw inputs if running in raw+derivatives mode
if derivatives and not opts.skip_bids_validation:
from fmripost_aroma.utils.bids import validate_input_dir

build_log.info(
Expand Down
4 changes: 4 additions & 0 deletions src/fmripost_aroma/workflows/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -210,6 +210,10 @@ def init_single_subject_wf(subject_id: str):
f"Please check your BIDS filters: {config.execution.bids_filters}."
)

config.loggers.workflow.info(
f'Collected subject data:\n{yaml.dump(subject_data, default_flow_style=False, indent=4)}',
)

bids_info = pe.Node(
BIDSInfo(
bids_dir=config.execution.bids_dir,
Expand Down

0 comments on commit df73880

Please sign in to comment.