Skip to content

Commit 9f7ae5d

Browse files
authored
Merge pull request #325 from Australian-Imaging-Service/develop
syntax updates to match pydra2app instead of arcana
2 parents e1d0912 + caaefb9 commit 9f7ae5d

File tree

8 files changed

+23
-28
lines changed

8 files changed

+23
-28
lines changed

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,6 @@ jobs:
5959
env:
6060
PUSH: "${{ steps.deployable.outputs.PUSH }}"
6161
run: >
62-
arcana deploy make-app ./australian-imaging-service xnat:XnatApp
62+
pydra2app make xnat ./australian-imaging-service
6363
--registry ghcr.io --check-registry --clean-up --tag-latest --loglevel info
6464
--release pipelines-metapackage $RELEASE $PUSH

australian-imaging-service/mri/human/neuro/bidsapps/fmriprep.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
title: 'Functional MRI data preprocessing pipeline'
2-
arcana_spec_version: 1.0
1+
title: "Functional MRI data preprocessing pipeline"
2+
schema_version: 1.0
33
version:
44
package: &package_version "23.1.4"
55
base_image:
@@ -110,7 +110,7 @@ command:
110110
configuration:
111111
path: fmap/magnitude
112112
column_defaults:
113-
datatype: medimage/dicom-series
113+
datatype: medimage/dicom-series
114114
fmap_phasediff:
115115
datatype: medimage/nifti-gz-x
116116
help: "Field map - BIDS Case 1: phasediff image corresponding to the phase-diff map between echo times"
@@ -171,5 +171,5 @@ command:
171171
app_output_dir: /work/bids-app-output
172172
app_work_dir: /work/nipype-work
173173
json_edits:
174-
path: 'fmap/.*'
174+
path: "fmap/.*"
175175
jq_expr: '.IntendedFor = "{bold}"'

australian-imaging-service/mri/human/neuro/bidsapps/mriqc.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
arcana_spec_version: 1.0
2-
title: 'Extract quality control metrics from T1w, T2W and fMRI data'
1+
schema_version: 1.0
2+
title: "Extract quality control metrics from T1w, T2W and fMRI data"
33
version:
44
package: &package_version "22.0.6"
55
build: "3"
@@ -11,7 +11,7 @@ base_image:
1111
tag: *package_version
1212
package_manager: apt
1313
docs:
14-
info_url: https://mriqc.readthedocs.io
14+
info_url: https://mriqc.readthedocs.io
1515
description: |
1616
MRIQC extracts no-reference IQMs (image quality metrics) from
1717
structural (T1w and T2w) and functional MRI (magnetic resonance

australian-imaging-service/mri/human/neuro/bidsapps/smriprep.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
title: "Structural MRI data preprocessing pipeline"
2-
arcana_spec_version: 1.0
2+
schema_version: 1.0
33
version:
44
package: &package_version "0.9.2"
55
build: "3"
@@ -69,4 +69,4 @@ command:
6969
executable: /opt/conda/bin/smriprep
7070
dataset: /work/bids-dataset
7171
app_output_dir: /work/bids-app-output
72-
app_work_dir: /work/nipype-work # We need to give a work directory outside of the pydra node as otherwise the file paths exceed Freesurfer's file-path char buffers
72+
app_work_dir: /work/nipype-work # We need to give a work directory outside of the pydra node as otherwise the file paths exceed Freesurfer's file-path char buffers

requirements.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
arcana >=0.10.11
2-
arcana-xnat >= 0.3.6
1+
pipeline2app >=0.12
2+
pipeline2app-xnat >= 0.6.2
33
fileformats >= 0.8.6
44
fileformats-extras >= 0.2.1
55
fileformats-datascience >= 0.1.0

scripts/debug_specific_case.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353
# image_tag=image_tag,
5454
# build_dir=build_dir,
5555
# use_local_packages=True,
56-
# use_test_config=True,
56+
# for_localhost=True,
5757
# license_src=license_src,
5858
# arcana_install_extras=["test"],
5959
# generate_only=True,

scripts/test_build.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,18 @@
11
from pathlib import Path
22
from click.testing import CliRunner
3-
from arcana.core.cli.deploy import make_app
4-
from arcana.core.utils.misc import show_cli_trace
3+
from pipeline2app.core.cli import make
4+
from frametree.core.utils import show_cli_trace
55

66
PKG_PATH = Path(__file__).parent.parent.absolute()
77

88
runner = CliRunner()
99

1010
results = runner.invoke(
11-
make_app,
11+
make,
1212
[
1313
f"{PKG_PATH}/australian-imaging-service/mri/human/neuro/bidsapps/fmriprep.yaml",
1414
"xnat:XnatApp",
15-
"--raise-errors"
15+
"--raise-errors",
1616
],
1717
catch_exceptions=False,
1818
)

tests/test_bids_apps.py

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import json
2-
from arcana.core.cli.deploy import make_app
2+
from arcana.core.cli.deploy import make
33
from arcana.core.utils.misc import show_cli_trace
44
from arcana.xnat.deploy import XnatApp
55
from arcana.xnat.utils.testing import install_and_launch_xnat_cs_command
@@ -24,14 +24,14 @@ def test_bids_app(
2424
build_arg = "--build"
2525

2626
result = cli_runner(
27-
make_app,
27+
make,
2828
[
2929
str(bp.spec_path),
3030
"pipelines-core-test",
3131
"--build-dir",
3232
str(build_dir),
3333
build_arg,
34-
"--use-test-config",
34+
"--for-localhost",
3535
"--use-local-packages",
3636
"--raise-errors",
3737
"--license-src",
@@ -46,17 +46,12 @@ def test_bids_app(
4646
with xnat_connect() as xlogin:
4747

4848
with open(
49-
build_dir
50-
/ image_spec.name
51-
/ "xnat_commands"
52-
/ (image_spec.name + ".json")
49+
build_dir / image_spec.name / "xnat_commands" / (image_spec.name + ".json")
5350
) as f:
5451
xnat_command = json.load(f)
5552
xnat_command.name = xnat_command.label = image_spec.name + run_prefix
5653

57-
test_xsession = next(
58-
iter(xlogin.projects[bp.project_id].experiments.values())
59-
)
54+
test_xsession = next(iter(xlogin.projects[bp.project_id].experiments.values()))
6055

6156
inputs_json = {}
6257

@@ -78,7 +73,7 @@ def test_bids_app(
7873
for pname, pval in bp.parameters.items():
7974
inputs_json[pname] = pval
8075

81-
inputs_json['Arcana_flags'] = (
76+
inputs_json["Arcana_flags"] = (
8277
"--plugin serial "
8378
"--work /work " # NB: work dir moved inside container due to file-locking issue on some mounted volumes (see https://github.com/tox-dev/py-filelock/issues/147)
8479
"--dataset-name default "

0 commit comments

Comments
 (0)