From 6795f96b93cc1b7243f9f403d6c3a022b70de6f2 Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:25:44 +0200 Subject: [PATCH 01/16] refactor: rename AmplitudeLowFrequencyFluctuationEstimator to ALFFEstimator --- junifer/markers/falff/falff_estimator.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/junifer/markers/falff/falff_estimator.py b/junifer/markers/falff/falff_estimator.py index 3455ace24..1bb60e19c 100644 --- a/junifer/markers/falff/falff_estimator.py +++ b/junifer/markers/falff/falff_estimator.py @@ -26,8 +26,8 @@ if TYPE_CHECKING: @singleton -class AmplitudeLowFrequencyFluctuationEstimator: - """Estimator class for AmplitudeLowFrequencyFluctuationBase. +class ALFFEstimator: + """Estimator class for (fractional) Amplitude Low Frequency Fluctuation. This class is a singleton and is used for efficient computation of fALFF, by caching the voxel-wise ALFF map for a given set of file path and -- 2.52.0 From a112b27fb4952d27473d45ec60fce19512f62792 Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:26:35 +0200 Subject: [PATCH 02/16] refactor: rename tests for AmplitudeLowFrequencyFluctuationEstimator to ALFFEstimator --- .../markers/falff/tests/test_falff_estimator.py | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/junifer/markers/falff/tests/test_falff_estimator.py b/junifer/markers/falff/tests/test_falff_estimator.py index bee2eaa1c..58f53391a 100644 --- a/junifer/markers/falff/tests/test_falff_estimator.py +++ b/junifer/markers/falff/tests/test_falff_estimator.py @@ -10,22 +10,20 @@ from nibabel import Nifti1Image from scipy.stats import pearsonr from junifer.datareader import DefaultDataReader -from junifer.markers.falff.falff_estimator import ( - AmplitudeLowFrequencyFluctuationEstimator, -) +from junifer.markers.falff.falff_estimator import ALFFEstimator from junifer.pipeline.utils import _check_afni from junifer.testing.datagrabbers import PartlyCloudyTestingDataGrabber from junifer.utils import logger -def test_AmplitudeLowFrequencyFluctuationEstimator_cache_python() -> None: +def test_ALFFEstimator_cache_python() -> None: """Test that the cache works properly when using python.""" with PartlyCloudyTestingDataGrabber() as dg: input = dg["sub-01"] input = DefaultDataReader().fit_transform(input) - estimator = AmplitudeLowFrequencyFluctuationEstimator() + estimator = ALFFEstimator() start_time = time.time() alff, falff = estimator.fit_transform( use_afni=False, @@ -111,14 +109,14 @@ def test_AmplitudeLowFrequencyFluctuationEstimator_cache_python() -> None: @pytest.mark.skipif( _check_afni() is False, reason="requires afni to be in PATH" ) -def test_AmplitudeLowFrequencyFluctuationEstimator_cache_afni() -> None: +def test_ALFFEstimator_cache_afni() -> None: """Test that the cache works properly when using afni.""" with PartlyCloudyTestingDataGrabber() as dg: input = dg["sub-01"] input = DefaultDataReader().fit_transform(input) - estimator = AmplitudeLowFrequencyFluctuationEstimator() + estimator = ALFFEstimator() start_time = time.time() alff, falff = estimator.fit_transform( use_afni=True, @@ -204,13 +202,13 @@ def test_AmplitudeLowFrequencyFluctuationEstimator_cache_afni() -> None: @pytest.mark.skipif( _check_afni() is False, reason="requires afni to be in PATH" ) -def test_AmplitudeLowFrequencyFluctuationEstimator_afni_vs_python() -> None: +def test_ALFFEstimator_afni_vs_python() -> None: """Test that the cache works properly when using afni.""" with PartlyCloudyTestingDataGrabber() as dg: input = dg["sub-01"] input = DefaultDataReader().fit_transform(input) - estimator = AmplitudeLowFrequencyFluctuationEstimator() + estimator = ALFFEstimator() # Use an arbitrary TR to test the AFNI vs Python implementation afni_alff, afni_falff = estimator.fit_transform( -- 2.52.0 From 115fe447a21b12863dab6238c9299fe91605c98d Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:27:06 +0200 Subject: [PATCH 03/16] refactor: rename AmplitudeLowFrequencyFluctuationBase to ALFFBase --- junifer/markers/falff/falff_base.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/junifer/markers/falff/falff_base.py b/junifer/markers/falff/falff_base.py index 1e581e39d..8b412ef21 100644 --- a/junifer/markers/falff/falff_base.py +++ b/junifer/markers/falff/falff_base.py @@ -10,10 +10,10 @@ from typing import Dict, List, Optional from ...utils.logging import raise_error from ..base import BaseMarker -from .falff_estimator import AmplitudeLowFrequencyFluctuationEstimator +from .falff_estimator import ALFFEstimator -class AmplitudeLowFrequencyFluctuationBase(BaseMarker): +class ALFFBase(BaseMarker): """Base class for (fractional) Amplitude Low Frequency Fluctuation. Parameters @@ -144,7 +144,7 @@ class AmplitudeLowFrequencyFluctuationBase(BaseMarker): "before calling the `compute` method." ) - estimator = AmplitudeLowFrequencyFluctuationEstimator() + estimator = ALFFEstimator() alff, falff = estimator.fit_transform( use_afni=self.use_afni, -- 2.52.0 From 3a45210c370f2755626b8716606c3a495d047e71 Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:27:32 +0200 Subject: [PATCH 04/16] refactor: rename AmplitudeLowFrequencyFluctuationParcels to ALFFEParcels --- junifer/markers/falff/falff_parcels.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/junifer/markers/falff/falff_parcels.py b/junifer/markers/falff/falff_parcels.py index 764731bab..2e02e368d 100644 --- a/junifer/markers/falff/falff_parcels.py +++ b/junifer/markers/falff/falff_parcels.py @@ -9,13 +9,11 @@ from typing import Dict, List, Optional, Union from ...api.decorators import register_marker from .. import ParcelAggregation -from .falff_base import AmplitudeLowFrequencyFluctuationBase +from .falff_base import ALFFBase @register_marker -class AmplitudeLowFrequencyFluctuationParcels( - AmplitudeLowFrequencyFluctuationBase -): +class ALFFParcels(ALFFBase): """Class for computing fALFF/ALFF on parcels. Parameters -- 2.52.0 From 269d03ddaf5ad838eaeffac8f9ff4f01461032ae Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:27:58 +0200 Subject: [PATCH 05/16] refactor: rename tests for AmplitudeLowFrequencyFluctuationParcels to ALFFEParcels --- .../markers/falff/tests/test_falff_parcels.py | 30 +++++++++---------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/junifer/markers/falff/tests/test_falff_parcels.py b/junifer/markers/falff/tests/test_falff_parcels.py index d1ccebbaf..50e52ddc8 100644 --- a/junifer/markers/falff/tests/test_falff_parcels.py +++ b/junifer/markers/falff/tests/test_falff_parcels.py @@ -11,7 +11,7 @@ from numpy.testing import assert_array_equal from scipy.stats import pearsonr from junifer.datareader import DefaultDataReader -from junifer.markers.falff import AmplitudeLowFrequencyFluctuationParcels +from junifer.markers.falff import ALFFParcels from junifer.pipeline.utils import _check_afni from junifer.storage import SQLiteFeatureStorage from junifer.testing.datagrabbers import PartlyCloudyTestingDataGrabber @@ -21,8 +21,8 @@ from junifer.utils import logger _PARCELLATION = "Schaefer100x7" -def test_AmplitudeLowFrequencyFluctuationParcels_python() -> None: - """Test AmplitudeLowFrequencyFluctuationParcels using python.""" +def test_ALFFParcels_python() -> None: + """Test ALFFParcels using python.""" # Get the SPM auditory data: with PartlyCloudyTestingDataGrabber() as dg: @@ -30,7 +30,7 @@ def test_AmplitudeLowFrequencyFluctuationParcels_python() -> None: input = DefaultDataReader().fit_transform(input) # Create ParcelAggregation object - marker = AmplitudeLowFrequencyFluctuationParcels( + marker = ALFFParcels( parcellation=_PARCELLATION, method="mean", use_afni=False, @@ -46,15 +46,15 @@ def test_AmplitudeLowFrequencyFluctuationParcels_python() -> None: @pytest.mark.skipif( _check_afni() is False, reason="requires afni to be in PATH" ) -def test_AmplitudeLowFrequencyFluctuationParcels_afni() -> None: - """Test AmplitudeLowFrequencyFluctuationParcels using afni.""" +def test_ALFFParcels_afni() -> None: + """Test ALFFParcels using afni.""" # Get the SPM auditory data: with PartlyCloudyTestingDataGrabber() as dg: input = dg["sub-01"] input = DefaultDataReader().fit_transform(input) # Create ParcelAggregation object - marker = AmplitudeLowFrequencyFluctuationParcels( + marker = ALFFParcels( parcellation=_PARCELLATION, method="mean", use_afni=True, @@ -67,7 +67,7 @@ def test_AmplitudeLowFrequencyFluctuationParcels_afni() -> None: assert afni_values.shape == (1, 100) # Again, should be blazing fast - marker = AmplitudeLowFrequencyFluctuationParcels( + marker = ALFFParcels( parcellation=_PARCELLATION, method="mean", fractional=False ) assert marker.use_afni is None @@ -82,10 +82,10 @@ def test_AmplitudeLowFrequencyFluctuationParcels_afni() -> None: @pytest.mark.parametrize( "fractional", [True, False], ids=["fractional", "non-fractional"] ) -def test_AmplitudeLowFrequencyFluctuationParcels_python_vs_afni( +def test_ALFFParcels_python_vs_afni( fractional: bool, ) -> None: - """Test AmplitudeLowFrequencyFluctuationParcels using python. + """Test ALFFParcels using python. Parameters ---------- @@ -98,7 +98,7 @@ def test_AmplitudeLowFrequencyFluctuationParcels_python_vs_afni( input = DefaultDataReader().fit_transform(input) # Create ParcelAggregation object - marker_python = AmplitudeLowFrequencyFluctuationParcels( + marker_python = ALFFParcels( parcellation=_PARCELLATION, method="mean", use_afni=False, @@ -110,7 +110,7 @@ def test_AmplitudeLowFrequencyFluctuationParcels_python_vs_afni( assert python_values.ndim == 2 assert python_values.shape == (1, 100) - marker_afni = AmplitudeLowFrequencyFluctuationParcels( + marker_afni = ALFFParcels( parcellation=_PARCELLATION, method="mean", use_afni=True, @@ -127,10 +127,10 @@ def test_AmplitudeLowFrequencyFluctuationParcels_python_vs_afni( assert r > 0.99 -def test_AmplitudeLowFrequencyFluctuationParcels_storage( +def test_ALFFParcels_storage( tmp_path: Path, ) -> None: - """Test AmplitudeLowFrequencyFluctuationParcels storage. + """Test ALFFParcels storage. Parameters ---------- @@ -142,7 +142,7 @@ def test_AmplitudeLowFrequencyFluctuationParcels_storage( input = dg["sub-01"] input = DefaultDataReader().fit_transform(input) # Create ParcelAggregation object - marker = AmplitudeLowFrequencyFluctuationParcels( + marker = ALFFParcels( parcellation=_PARCELLATION, method="mean", use_afni=False, -- 2.52.0 From 385ee6f34161e65ba88616ed2ffcb7c4685d3ce6 Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:29:41 +0200 Subject: [PATCH 06/16] refactor: rename AmplitudeLowFrequencyFluctuationSpheres to ALFFESpheres --- junifer/markers/falff/falff_spheres.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/junifer/markers/falff/falff_spheres.py b/junifer/markers/falff/falff_spheres.py index 574fa66f3..2d38a195d 100644 --- a/junifer/markers/falff/falff_spheres.py +++ b/junifer/markers/falff/falff_spheres.py @@ -9,13 +9,11 @@ from typing import Dict, List, Optional, Union from ...api.decorators import register_marker from .. import SphereAggregation -from .falff_base import AmplitudeLowFrequencyFluctuationBase +from .falff_base import ALFFBase @register_marker -class AmplitudeLowFrequencyFluctuationSpheres( - AmplitudeLowFrequencyFluctuationBase -): +class ALFFSpheres(ALFFBase): """Class for computing fALFF/ALFF on spheres. Parameters -- 2.52.0 From a9dfdd67e1a19b4c71520c5bd30bdfedfc1b2842 Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:30:24 +0200 Subject: [PATCH 07/16] refactor: rename tests for AmplitudeLowFrequencyFluctuationSpheres to ALFFESpheres --- .../markers/falff/tests/test_falff_spheres.py | 30 +++++++++---------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/junifer/markers/falff/tests/test_falff_spheres.py b/junifer/markers/falff/tests/test_falff_spheres.py index b1e9d8df7..556cd3844 100644 --- a/junifer/markers/falff/tests/test_falff_spheres.py +++ b/junifer/markers/falff/tests/test_falff_spheres.py @@ -11,7 +11,7 @@ from numpy.testing import assert_array_equal from scipy.stats import pearsonr from junifer.datareader import DefaultDataReader -from junifer.markers.falff import AmplitudeLowFrequencyFluctuationSpheres +from junifer.markers.falff import ALFFSpheres from junifer.pipeline.utils import _check_afni from junifer.storage import SQLiteFeatureStorage from junifer.testing.datagrabbers import PartlyCloudyTestingDataGrabber @@ -21,8 +21,8 @@ from junifer.utils import logger _COORDINATES = "DMNBuckner" -def test_AmplitudeLowFrequencyFluctuationSpheres_python() -> None: - """Test AmplitudeLowFrequencyFluctuationSpheres using python.""" +def test_ALFFSpheres_python() -> None: + """Test ALFFSpheres using python.""" # Get the SPM auditory data: with PartlyCloudyTestingDataGrabber() as dg: @@ -30,7 +30,7 @@ def test_AmplitudeLowFrequencyFluctuationSpheres_python() -> None: input = DefaultDataReader().fit_transform(input) # Create ParcelAggregation object - marker = AmplitudeLowFrequencyFluctuationSpheres( + marker = ALFFSpheres( coords=_COORDINATES, radius=5, method="mean", @@ -47,15 +47,15 @@ def test_AmplitudeLowFrequencyFluctuationSpheres_python() -> None: @pytest.mark.skipif( _check_afni() is False, reason="requires afni to be in PATH" ) -def test_AmplitudeLowFrequencyFluctuationSpheres_afni() -> None: - """Test AmplitudeLowFrequencyFluctuationSpheres using afni.""" +def test_ALFFSpheres_afni() -> None: + """Test ALFFSpheres using afni.""" # Get the SPM auditory data: with PartlyCloudyTestingDataGrabber() as dg: input = dg["sub-01"] input = DefaultDataReader().fit_transform(input) # Create ParcelAggregation object - marker = AmplitudeLowFrequencyFluctuationSpheres( + marker = ALFFSpheres( coords=_COORDINATES, radius=5, method="mean", @@ -69,7 +69,7 @@ def test_AmplitudeLowFrequencyFluctuationSpheres_afni() -> None: assert afni_values.shape == (1, 6) # Again, should be blazing fast - marker = AmplitudeLowFrequencyFluctuationSpheres( + marker = ALFFSpheres( coords=_COORDINATES, radius=5, method="mean", @@ -87,10 +87,10 @@ def test_AmplitudeLowFrequencyFluctuationSpheres_afni() -> None: @pytest.mark.parametrize( "fractional", [True, False], ids=["fractional", "non-fractional"] ) -def test_AmplitudeLowFrequencyFluctuationSpheres_python_vs_afni( +def test_ALFFSpheres_python_vs_afni( fractional: bool, ) -> None: - """Test AmplitudeLowFrequencyFluctuationSpheres python vs afni results. + """Test ALFFSpheres python vs afni results. Parameters ---------- @@ -102,7 +102,7 @@ def test_AmplitudeLowFrequencyFluctuationSpheres_python_vs_afni( input = DefaultDataReader().fit_transform(input) # Create ParcelAggregation object - marker_python = AmplitudeLowFrequencyFluctuationSpheres( + marker_python = ALFFSpheres( coords=_COORDINATES, radius=5, method="mean", @@ -115,7 +115,7 @@ def test_AmplitudeLowFrequencyFluctuationSpheres_python_vs_afni( assert python_values.ndim == 2 assert python_values.shape == (1, 6) - marker_afni = AmplitudeLowFrequencyFluctuationSpheres( + marker_afni = ALFFSpheres( coords=_COORDINATES, radius=5, method="mean", @@ -133,10 +133,10 @@ def test_AmplitudeLowFrequencyFluctuationSpheres_python_vs_afni( assert r > 0.99 -def test_AmplitudeLowFrequencyFluctuationSpheres_storage( +def test_ALFFSpheres_storage( tmp_path: Path, ) -> None: - """Test AmplitudeLowFrequencyFluctuationSpheres storage. + """Test ALFFSpheres storage. Parameters ---------- @@ -148,7 +148,7 @@ def test_AmplitudeLowFrequencyFluctuationSpheres_storage( input = dg["sub-01"] input = DefaultDataReader().fit_transform(input) # Create ParcelAggregation object - marker = AmplitudeLowFrequencyFluctuationSpheres( + marker = ALFFSpheres( coords=_COORDINATES, radius=5, method="mean", -- 2.52.0 From 35d1e7b757e28ddf7a2b336bc12526505a6ac24a Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:30:34 +0200 Subject: [PATCH 08/16] refactor: rename imports in junifer.markers.falff.__init__ --- junifer/markers/falff/__init__.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/junifer/markers/falff/__init__.py b/junifer/markers/falff/__init__.py index 6e1023665..c04f42aa9 100644 --- a/junifer/markers/falff/__init__.py +++ b/junifer/markers/falff/__init__.py @@ -3,5 +3,5 @@ # Authors: Federico Raimondo # License: AGPL -from .falff_parcels import AmplitudeLowFrequencyFluctuationParcels -from .falff_spheres import AmplitudeLowFrequencyFluctuationSpheres +from .falff_parcels import ALFFParcels +from .falff_spheres import ALFFSpheres -- 2.52.0 From 778ea5ea62384951f49514901e48523579bc1bf2 Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:31:15 +0200 Subject: [PATCH 09/16] refactor: rename imports in junifer.markers.__init__ --- junifer/markers/__init__.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/junifer/markers/__init__.py b/junifer/markers/__init__.py index 26f259edf..d6c67d4e6 100644 --- a/junifer/markers/__init__.py +++ b/junifer/markers/__init__.py @@ -18,10 +18,7 @@ from .functional_connectivity import ( EdgeCentricFCSpheres, ) from .reho import ReHoParcels, ReHoSpheres -from .falff import ( - AmplitudeLowFrequencyFluctuationParcels, - AmplitudeLowFrequencyFluctuationSpheres, -) +from .falff import ALFFParcels, ALFFSpheres from .temporal_snr import ( TemporalSNRParcels, TemporalSNRSpheres, -- 2.52.0 From ee21edb53ce9c0118b756c44beeea3402b3a34c7 Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:31:25 +0200 Subject: [PATCH 10/16] chore: isort --- junifer/markers/parcel_aggregation.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/junifer/markers/parcel_aggregation.py b/junifer/markers/parcel_aggregation.py index c8c3f892c..20d0a7257 100644 --- a/junifer/markers/parcel_aggregation.py +++ b/junifer/markers/parcel_aggregation.py @@ -13,7 +13,7 @@ from nilearn.maskers import NiftiMasker from ..api.decorators import register_marker from ..data import get_mask, load_parcellation, merge_parcellations from ..stats import get_aggfunc_by_name -from ..utils import logger, warn_with_log, raise_error +from ..utils import logger, raise_error, warn_with_log from .base import BaseMarker -- 2.52.0 From c5399e6b204e3b3208ff0f661ee63b447fc8a38e Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:36:12 +0200 Subject: [PATCH 11/16] chore: add changelog 216.change --- docs/changes/newsfragments/216.change | 1 + 1 file changed, 1 insertion(+) create mode 100644 docs/changes/newsfragments/216.change diff --git a/docs/changes/newsfragments/216.change b/docs/changes/newsfragments/216.change new file mode 100644 index 000000000..01d731587 --- /dev/null +++ b/docs/changes/newsfragments/216.change @@ -0,0 +1 @@ +``AmplitudeLowFrequencyFluctuationParcels`` and ``AmplitudeLowFrequencyFluctuationSpheres`` are renamed to :class:`junifer.markers.ALFFParcels` and :class:`junifer.markers.ALFFSpheres` by `Synchon Mandal`_ \ No newline at end of file -- 2.52.0 From e008cda7ae0a1de8c385c59a9974e0c40cc56ce8 Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:40:56 +0200 Subject: [PATCH 12/16] docs: rename instances of AmplitudeLowFrequencyFluctuation to ALFF in whats_new.rst --- docs/whats_new.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/whats_new.rst b/docs/whats_new.rst index a6b9e50a8..717d3c5e6 100644 --- a/docs/whats_new.rst +++ b/docs/whats_new.rst @@ -150,8 +150,8 @@ Features - Implement :class:`junifer.markers.ReHoParcels` and :class:`junifer.markers.ReHoSpheres` markers by `Synchon Mandal`_ (:gh:`36`) -- Implement :class:`junifer.markers.AmplitudeLowFrequencyFluctuationParcels` and - :class:`junifer.markers.AmplitudeLowFrequencyFluctuationSpheres` markers by +- Implement ``junifer.markers.AmplitudeLowFrequencyFluctuationParcels`` and + ``junifer.markers.AmplitudeLowFrequencyFluctuationSpheres`` markers by `Fede Raimondo`_ (:gh:`35`) Misc -- 2.52.0 From d36e527ecb93e164a3cfedff2d50c3d66c9c419e Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:41:59 +0200 Subject: [PATCH 13/16] docs: rename instances of AmplitudeLowFrequencyFluctuation to ALFF in builtin.rst --- docs/builtin.rst | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/docs/builtin.rst b/docs/builtin.rst index 0913a0c78..2ec046296 100644 --- a/docs/builtin.rst +++ b/docs/builtin.rst @@ -176,11 +176,11 @@ Available - Calculate regional homogeneity over spheres placed on coordinates - Done - 0.0.1 - * - :class:`junifer.markers.AmplitudeLowFrequencyFluctuationParcels` + * - :class:`junifer.markers.ALFFParcels` - Calculate (f)ALFF and aggregate using parcellations - Done - 0.0.1 - * - :class:`junifer.markers.AmplitudeLowFrequencyFluctuationSpheres` + * - :class:`junifer.markers.ALFFSpheres` - Calculate (f)ALFF and aggregate using spheres placed on coordinates - Done - 0.0.1 @@ -541,13 +541,13 @@ Available * - Nilearn's MNI152 1mm-resolution mask - | ``compute_brain_mask`` - 0.0.2 - - | Compute the whole-brain mask. This mask is calculated using MNI152 1mm-resolution template mask onto the + - | Compute the whole-brain mask. This mask is calculated using MNI152 1mm-resolution template mask onto the | target image. See :func:`nilearn.masking.compute_brain_mask` * - Nilearn's mask computed from FMRI data - | ``compute_epi_mask`` - 0.0.2 - - | Compute a brain mask from fMRI data. This is based on an heuristic proposed by T.Nichols: find the least - | dense point of the histogram, between fractions ``lower_cutoff`` and ``upper_cutoff`` of the total image + - | Compute a brain mask from fMRI data. This is based on an heuristic proposed by T.Nichols: find the least + | dense point of the histogram, between fractions ``lower_cutoff`` and ``upper_cutoff`` of the total image | histogram. See :func:`nilearn.masking.compute_epi_mask` * - Nilearn's background mask - | ``compute_background_mask`` -- 2.52.0 From 24db2704689088673aa26910e688f50b6c144b9c Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 14:42:12 +0200 Subject: [PATCH 14/16] chore: rename remaining instance of AmplitudeLowFrequencyFluctuation in ALFFEstimator --- junifer/markers/falff/falff_estimator.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/junifer/markers/falff/falff_estimator.py b/junifer/markers/falff/falff_estimator.py index 1bb60e19c..e1070f1af 100644 --- a/junifer/markers/falff/falff_estimator.py +++ b/junifer/markers/falff/falff_estimator.py @@ -34,8 +34,7 @@ class ALFFEstimator: computation parameters. .. warning:: This class can only be used via - :class:`junifer.markers.falff.AmplitudeLowFrequencyFluctuationBase` - as it serves a specific purpose. + :class:`junifer.markers.falff.ALFFBase` as it serves a specific purpose. Parameters ---------- -- 2.52.0 From 50553a07c527662558e1cad6cdf37af8734c0260 Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 17:29:21 +0200 Subject: [PATCH 15/16] docs: use proper references for ALFF markers in whats_new.rst --- docs/whats_new.rst | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/docs/whats_new.rst b/docs/whats_new.rst index 717d3c5e6..4434cfc72 100644 --- a/docs/whats_new.rst +++ b/docs/whats_new.rst @@ -150,9 +150,8 @@ Features - Implement :class:`junifer.markers.ReHoParcels` and :class:`junifer.markers.ReHoSpheres` markers by `Synchon Mandal`_ (:gh:`36`) -- Implement ``junifer.markers.AmplitudeLowFrequencyFluctuationParcels`` and - ``junifer.markers.AmplitudeLowFrequencyFluctuationSpheres`` markers by - `Fede Raimondo`_ (:gh:`35`) +- Implement :class:`junifer.markers.ALFFParcels` and + :class:`junifer.markers.ALFFSpheres` markers by `Fede Raimondo`_ (:gh:`35`) Misc ^^^^ -- 2.52.0 From c2de9f612614746c855c1e2e688d897a65bbb3be Mon Sep 17 00:00:00 2001 From: Synchon Mandal Date: Thu, 30 Mar 2023 17:31:33 +0200 Subject: [PATCH 16/16] chore: fix ALFF references in 187.bugfix --- docs/changes/newsfragments/187.bugfix | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/changes/newsfragments/187.bugfix b/docs/changes/newsfragments/187.bugfix index 3ddfdd4c7..4c852e949 100644 --- a/docs/changes/newsfragments/187.bugfix +++ b/docs/changes/newsfragments/187.bugfix @@ -1 +1 @@ -Fix :class:`junifer.markers.AmplitudeLowFrequencyFluctuationParcels`, :class:`junifer.markers.AmplitudeLowFrequencyFluctuationSpheres`, :class:`junifer.markers.ReHoSpheres` and :class:`junifer.markers.ReHoParcels` pass the ``extra_input`` parameter by `Fede Raimondo`_ \ No newline at end of file +Fix :class:`junifer.markers.ALFFParcels`, :class:`junifer.markers.ALFFSpheres`, :class:`junifer.markers.ReHoSpheres` and :class:`junifer.markers.ReHoParcels` pass the ``extra_input`` parameter by `Fede Raimondo`_ \ No newline at end of file -- 2.52.0