Skip to content

Commit

Permalink
change structure
Browse files Browse the repository at this point in the history
  • Loading branch information
thibaultdvx committed Nov 18, 2024
1 parent e69fdc0 commit 59a7c55
Show file tree
Hide file tree
Showing 19 changed files with 53 additions and 48 deletions.
File renamed without changes.
File renamed without changes.
1 change: 1 addition & 0 deletions src/clinicaio/generators/caps/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
from .generator import CAPSGenerator
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
from pathlib import Path
from typing import Optional

from clinicaio.utils.bids_entities import (
from clinicaio.models.bids_entities import (
DescriptionEntity,
ResolutionEntity,
SessionEntity,
Expand All @@ -10,10 +10,10 @@
SUVREntity,
TracerEntity,
)
from clinicaio.utils.caps import Description, Extension, Suffix
from clinicaio.models.caps import Description, Extension, Suffix


def get_caps_filename(
def _get_caps_filename(
subject: int,
session: int,
suffix: str,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,13 @@
import numpy as np
from scipy.io import savemat

from clinicaio.utils.bids_entities import SessionEntity, SubjectEntity
from clinicaio.utils.caps import Extension, Resolution, Space, Suffix
from clinicaio.models.bids_entities import SessionEntity, SubjectEntity
from clinicaio.models.caps import Extension, Resolution, Space, Suffix

from .filename import get_caps_filename
from .filename import _get_caps_filename


def build_flair_linear(
def _build_flair_linear(
root: Union[str, Path], subject: int, session: int, crop: bool = True
):
"""
Expand All @@ -34,7 +34,7 @@ def build_flair_linear(
)
directory.mkdir(parents=True, exist_ok=True)

uncropped_file = directory / get_caps_filename(
uncropped_file = directory / _get_caps_filename(
subject,
session,
space=space,
Expand All @@ -44,7 +44,7 @@ def build_flair_linear(
)
nib.save(dummy_nifti_img, uncropped_file)

mat_file = directory / get_caps_filename(
mat_file = directory / _get_caps_filename(
subject,
session,
space=space,
Expand All @@ -55,7 +55,7 @@ def build_flair_linear(
savemat(mat_file, dummy_mat)

if crop:
cropped_file = directory / get_caps_filename(
cropped_file = directory / _get_caps_filename(
subject,
session,
space=space,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,12 @@
from pathlib import Path
from typing import List, Union

from clinicaio.utils.bids_entities import SessionEntity, SubjectEntity
from clinicaio.utils.caps import Pipeline
from clinicaio.models.bids_entities import SessionEntity, SubjectEntity
from clinicaio.models.caps import Pipeline

from .flair_linear import build_flair_linear
from .pet_linear import build_pet_linear
from .t1_linear import build_t1_linear
from .flair_linear import _build_flair_linear
from .pet_linear import _build_pet_linear
from .t1_linear import _build_t1_linear


class CAPSGenerator:
Expand Down Expand Up @@ -57,11 +57,11 @@ def build_pipeline(
"""
pipeline = Pipeline(pipeline)
if pipeline == Pipeline.T1_LINEAR:
builder = build_t1_linear
builder = _build_t1_linear
elif pipeline == Pipeline.FLAIR_LINEAR:
builder = build_flair_linear
builder = _build_flair_linear
elif pipeline == Pipeline.PET_LINEAR:
builder = build_pet_linear
builder = _build_pet_linear
else:
raise ValueError(f"pipeline {pipeline} is not yet implemented.")

Expand Down Expand Up @@ -101,9 +101,13 @@ def remove_pipeline(
raise ValueError(f"pipeline {pipeline} is not yet implemented.")

full_dir = (
self.dir / "subjects" / SubjectEntity(subject) / SessionEntity(session) / directory
self.dir
/ "subjects"
/ SubjectEntity(subject)
/ SessionEntity(session)
/ directory
)
try:
shutil.rmtree(full_dir)
except FileNotFoundError: # there is not such subject/session for this pipeline
except FileNotFoundError: # there is not such subject/session for this pipeline
pass
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,14 @@
import numpy as np
from scipy.io import savemat

from clinicaio.utils.bids_entities import SessionEntity, SubjectEntity
from clinicaio.utils.caps import Extension, Resolution, Space, Suffix
from clinicaio.utils.pet import SUVRReferenceRegion, Tracer
from clinicaio.models.bids_entities import SessionEntity, SubjectEntity
from clinicaio.models.caps import Extension, Resolution, Space, Suffix
from clinicaio.models.pet import SUVRReferenceRegion, Tracer

from .filename import get_caps_filename
from .filename import _get_caps_filename


def build_pet_linear(
def _build_pet_linear(
root: Union[str, Path],
subject: int,
session: int,
Expand Down Expand Up @@ -42,7 +42,7 @@ def build_pet_linear(
)
directory.mkdir(parents=True, exist_ok=True)

uncropped_file = directory / get_caps_filename(
uncropped_file = directory / _get_caps_filename(
subject,
session,
tracer=trc,
Expand All @@ -55,7 +55,7 @@ def build_pet_linear(
)
nib.save(dummy_nifti_img, uncropped_file)

mat_file = directory / get_caps_filename(
mat_file = directory / _get_caps_filename(
subject,
session,
tracer=trc,
Expand All @@ -66,7 +66,7 @@ def build_pet_linear(
savemat(mat_file, dummy_mat)

if crop:
cropped_file = directory / get_caps_filename(
cropped_file = directory / _get_caps_filename(
subject,
session,
tracer=trc,
Expand All @@ -80,7 +80,7 @@ def build_pet_linear(
nib.save(dummy_nifti_img, cropped_file)

if save_pet_in_t1w_space:
nifti_file = directory / get_caps_filename(
nifti_file = directory / _get_caps_filename(
subject,
session,
tracer=trc,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,13 @@
import numpy as np
from scipy.io import savemat

from clinicaio.utils.bids_entities import SessionEntity, SubjectEntity
from clinicaio.utils.caps import Extension, Resolution, Space, Suffix
from clinicaio.models.bids_entities import SessionEntity, SubjectEntity
from clinicaio.models.caps import Extension, Resolution, Space, Suffix

from .filename import get_caps_filename
from .filename import _get_caps_filename


def build_t1_linear(
def _build_t1_linear(
root: Union[str, Path], subject: int, session: int, crop: bool = True
):
"""
Expand All @@ -34,7 +34,7 @@ def build_t1_linear(
)
directory.mkdir(parents=True, exist_ok=True)

uncropped_file = directory / get_caps_filename(
uncropped_file = directory / _get_caps_filename(
subject,
session,
space=space,
Expand All @@ -44,7 +44,7 @@ def build_t1_linear(
)
nib.save(dummy_nifti_img, uncropped_file)

mat_file = directory / get_caps_filename(
mat_file = directory / _get_caps_filename(
subject,
session,
space=space,
Expand All @@ -55,7 +55,7 @@ def build_t1_linear(
savemat(mat_file, dummy_mat)

if crop:
cropped_file = directory / get_caps_filename(
cropped_file = directory / _get_caps_filename(
subject,
session,
space=space,
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
Empty file.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

import pytest

from clinicaio.utils.caps_generator.filename import get_caps_filename
from clinicaio.generators.caps.filename import _get_caps_filename


@pytest.mark.parametrize(
Expand Down Expand Up @@ -38,6 +38,6 @@
],
)
def test_get_caps_filename(args, expected_output):
filename = get_caps_filename(**args)
filename = _get_caps_filename(**args)
assert isinstance(filename, Path)
assert str(filename) == expected_output
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import shutil
from pathlib import Path

from clinicaio.utils.caps_generator.flair_linear import build_flair_linear
from clinicaio.generators.caps.flair_linear import _build_flair_linear


def test_build_flair_linear():
Expand All @@ -11,7 +11,7 @@ def test_build_flair_linear():
if test_dir.exists():
shutil.rmtree(test_dir)

build_flair_linear(root=test_dir, subject=2, session=3)
_build_flair_linear(root=test_dir, subject=2, session=3)
assert (
pipeline_dir
/ "sub-002_ses-M003_space-MNI152NLin2009cSym_desc-Crop_res-1x1x1_FLAIR.nii.gz"
Expand All @@ -26,7 +26,7 @@ def test_build_flair_linear():

shutil.rmtree(test_dir)

build_flair_linear(root=test_dir, subject=2, session=3, crop=False)
_build_flair_linear(root=test_dir, subject=2, session=3, crop=False)
assert not (
pipeline_dir
/ "sub-002_ses-M003_space-MNI152NLin2009cSym_desc-Crop_res-1x1x1_FLAIR.nii.gz"
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import shutil
from pathlib import Path

from clinicaio.utils.caps_generator.generator import CAPSGenerator
from clinicaio.generators.caps.generator import CAPSGenerator


def test_caps_generator():
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import shutil
from pathlib import Path

from clinicaio.utils.caps_generator.pet_linear import build_pet_linear
from clinicaio.generators.caps.pet_linear import _build_pet_linear


def test_build_pet_linear():
Expand All @@ -11,7 +11,7 @@ def test_build_pet_linear():
if test_dir.exists():
shutil.rmtree(test_dir)

build_pet_linear(
_build_pet_linear(
root=test_dir,
subject=2,
session=3,
Expand All @@ -32,7 +32,7 @@ def test_build_pet_linear():

shutil.rmtree(test_dir)

build_pet_linear(
_build_pet_linear(
root=test_dir,
subject=2,
session=3,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import shutil
from pathlib import Path

from clinicaio.utils.caps_generator.t1_linear import build_t1_linear
from clinicaio.generators.caps.t1_linear import _build_t1_linear


def test_build_t1_linear():
Expand All @@ -11,7 +11,7 @@ def test_build_t1_linear():
if test_dir.exists():
shutil.rmtree(test_dir)

build_t1_linear(root=test_dir, subject=2, session=3, crop=True)
_build_t1_linear(root=test_dir, subject=2, session=3, crop=True)
assert (
pipeline_dir
/ "sub-002_ses-M003_space-MNI152NLin2009cSym_desc-Crop_res-1x1x1_T1w.nii.gz"
Expand All @@ -25,7 +25,7 @@ def test_build_t1_linear():

shutil.rmtree(test_dir)

build_t1_linear(root=test_dir, subject=2, session=3, crop=False)
_build_t1_linear(root=test_dir, subject=2, session=3, crop=False)
assert not (
pipeline_dir
/ "sub-002_ses-M003_space-MNI152NLin2009cSym_desc-Crop_res-1x1x1_T1w.nii.gz"
Expand Down

0 comments on commit 59a7c55

Please sign in to comment.