nimare 0.5.3rc1__py3-none-any.whl → 0.5.4__py3-none-any.whl

nimare/_version.py

@@ -8,11 +8,11 @@ import json
 
 version_json = '''
 {
- "date": "2025-07-26T17:24:25-0500",
+ "date": "2025-07-30T11:49:13-0500",
  "dirty": false,
  "error": null,
- "full-revisionid": "096951f4879c2b5f508463c2efc4a3da9ebaf9bc",
- "version": "0.5.3rc1"
+ "full-revisionid": "aa29fabbe739d247d1502bd7825a670018651aa5",
+ "version": "0.5.4"
 }
 '''  # END VERSION_JSON
 

nimare/io.py

@@ -63,31 +63,53 @@ def convert_nimads_to_dataset(studyset, annotation=None):
             },
         }
 
-        sample_sizes = analysis.metadata.get("sample_sizes")
-        sample_size = None
+        sample_sizes = analysis.metadata.get("sample_sizes", None)
+        sample_size = analysis.metadata.get("sample_size", None)
 
         # Validate sample sizes if present
         if sample_sizes is not None and not isinstance(sample_sizes, (list, tuple)):
-            raise TypeError(
-                f"Expected sample_sizes to be list or tuple, but got {type(sample_sizes)}"
+            LGR.warning(
+                f"Expected sample_sizes to be list or tuple, but got {type(sample_sizes)}."
             )
-
-        if not sample_sizes:
-            # Try to get single sample size from analysis or study metadata
-            sample_size = analysis.metadata.get("sample_size")
-            if sample_size is None:
-                sample_size = study.metadata.get("sample_size")
-
+            sample_sizes = None
+        elif sample_sizes is not None:
+            # Validate each sample size in the list
+            for i, ss in enumerate(sample_sizes):
+                if not isinstance(ss, (int, float)):
+                    LGR.warning(
+                        f"Expected sample_sizes[{i}] to be numeric, but got {type(ss)}."
+                        " Attempting to convert to numeric."
+                    )
+                try:
+                    sample_sizes[i] = int(ss)
+                except (ValueError, TypeError):
+                    try:
+                        sample_sizes[i] = float(ss)
+                    except (ValueError, TypeError):
+                        LGR.warning(f"Could not convert {ss} to numeric from type {type(ss)}.")
+                        sample_sizes = None
+                        break
+
+        if not sample_sizes and sample_size:
             # Validate single sample size if present
-            if sample_size is not None and not isinstance(sample_size, (int, float)):
-                raise TypeError(f"Expected sample_size to be numeric, but got {type(sample_size)}")
-
-        # Add sample size info to result if available
-        if sample_sizes or sample_size is not None:
+            if not isinstance(sample_size, (int, float)):
+                LGR.warning(
+                    f"Expected sample_size to be numeric, but got {type(sample_size)}."
+                    " Attempting to convert to numeric."
+                )
             try:
-                result["metadata"]["sample_sizes"] = sample_sizes or [sample_size]
-            except TypeError as e:
-                raise TypeError(f"Error converting sample size data to list: {str(e)}") from e
+                sample_sizes = [int(sample_size)]
+            except (ValueError, TypeError):
+                try:
+                    sample_sizes = [float(sample_size)]
+                except (ValueError, TypeError):
+                    LGR.warning(
+                        f"Could not convert {sample_size} to"
+                        f" numeric from type {type(sample_size)}."
+                    )
+                    sample_sizes = None
+        if sample_sizes:
+            result["metadata"]["sample_sizes"] = sample_sizes
 
         # Handle annotations if present
         if analysis.annotations:

nimare/tests/test_io.py

@@ -1,5 +1,6 @@
 """Test nimare.io (Dataset IO/transformations)."""
 
+import copy
 import os
 
 import pytest
@@ -52,16 +53,107 @@ def test_convert_nimads_to_dataset_single_sample_size(
     assert "sample_sizes" in dset.metadata.columns
 
 
-def test_analysis_to_dict_invalid_sample_sizes_type(example_nimads_studyset):
-    """Test _analysis_to_dict raises ValueError when sample_sizes is not a list/tuple."""
+@pytest.mark.parametrize(
+    "sample_sizes_val,sample_size_val,expect_col,expect_warning",
+    [
+        (5, None, False, True),
+        ([5, "6", "7.3", "not_a_number"], None, False, True),
+        (None, 10, True, False),
+        (None, 10.5, True, False),
+        (None, "12", True, True),
+        (None, "13.5", True, True),
+        (None, "not_a_number", False, True),
+        (None, None, False, False),
+        ([], 7, True, False),
+        ([], None, False, False),
+    ],
+)
+def test_analysis_to_dict_sample_size(
+    example_nimads_studyset, sample_sizes_val, sample_size_val, expect_col, expect_warning, caplog
+):
+    """Test conversion of nimads JSON to nimare dataset with different sample_size(s) values."""
     studyset = Studyset(example_nimads_studyset)
-    # Set sample_sizes to an int rather than list/tuple
     for study in studyset.studies:
         for analysis in study.analyses:
-            analysis.metadata["sample_sizes"] = 5
-    with pytest.raises(TypeError):
-        # Trigger conversion which internally calls _analysis_to_dict
-        io.convert_nimads_to_dataset(studyset)
+            analysis.metadata.clear()
+            if sample_sizes_val is not None:
+                analysis.metadata["sample_sizes"] = sample_sizes_val
+            if sample_size_val is not None:
+                analysis.metadata["sample_size"] = sample_size_val
+
+    with caplog.at_level("WARNING"):
+        dset = io.convert_nimads_to_dataset(studyset)
+    assert isinstance(dset, nimare.dataset.Dataset)
+    if expect_col:
+        assert "sample_sizes" in dset.metadata.columns
+    else:
+        assert "sample_sizes" not in dset.metadata.columns
+    if expect_warning:
+        assert any(
+            "sample_size" in record.message or "sample_sizes" in record.message
+            for record in caplog.records
+        )
+    else:
+        assert not any(
+            "sample_size" in record.message or "sample_sizes" in record.message
+            for record in caplog.records
+        )
+
+
+@pytest.mark.parametrize(
+    "annotation_mod,expect_success,expect_typeerror",
+    [
+        # No annotation at all
+        (None, True, False),
+        # Annotation with empty notes
+        (lambda ann: ann.update({"notes": []}), True, False),
+        # Annotation with extra irrelevant key
+        (lambda ann: ann.update({"extra_key": 123}), True, False),
+        # Annotation with missing 'notes' key (should fail)
+        (lambda ann: ann.pop("notes", None), False, True),
+        # Annotation with mismatched analysis id in notes (should warn/fail)
+        (
+            lambda ann: ann["notes"].append(
+                {
+                    "analysis_name": "Fake",
+                    "publication": "Fake",
+                    "study": ann["notes"][0]["study"],
+                    "study_year": 2025,
+                    "analysis": "not_in_studyset",
+                    "authors": "Nobody",
+                    "note": {"include": False},
+                    "study_name": "Fake",
+                }
+            ),
+            True,
+            True,
+        ),
+    ],
+)
+def test_analysis_to_dict_annotation(
+    example_nimads_studyset,
+    example_nimads_annotation,
+    annotation_mod,
+    expect_success,
+    expect_typeerror,
+):
+    """Test conversion of nimads JSON to nimare dataset with various annotation modifications."""
+    studyset = Studyset(example_nimads_studyset)
+    if annotation_mod is not None:
+        annotation = copy.deepcopy(example_nimads_annotation)
+        annotation_mod(annotation)
+        if expect_typeerror:
+            with pytest.raises((TypeError, ValueError, KeyError)):
+                studyset.annotations = annotation
+                io.convert_nimads_to_dataset(studyset)
+        else:
+            studyset.annotations = annotation
+            dset = io.convert_nimads_to_dataset(studyset)
+            assert expect_success
+    else:
+        # No annotation
+        dset = io.convert_nimads_to_dataset(studyset)
+        assert isinstance(dset, nimare.dataset.Dataset)
 
 
 def test_convert_sleuth_to_dataset_smoke():

{nimare-0.5.3rc1.dist-info → nimare-0.5.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: nimare
-Version: 0.5.3rc1
+Version: 0.5.4
 Summary: NiMARE: Neuroimaging Meta-Analysis Research Environment
 Home-page: https://github.com/neurostuff/NiMARE
 Author: NiMARE developers

{nimare-0.5.3rc1.dist-info → nimare-0.5.4.dist-info}/RECORD

@@ -1,7 +1,7 @@
 benchmarks/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 benchmarks/bench_cbma.py,sha256=fg_EER8hohi6kD1Hno_uXsFntKuCLTvseu-6OqkHkoU,1382
 nimare/__init__.py,sha256=HHIq3EimSZSf3zJSDwuTjBig1GbRwHGYfRLSqI3yleE,802
-nimare/_version.py,sha256=cTtTgV3QoHjAwvIOfB-51ttLjBig-3By0hQuB3n913w,500
+nimare/_version.py,sha256=tVBIpdAoja0KoEWsIhRcTCMaFKEJBHi8NwQmlr5NZqw,497
 nimare/base.py,sha256=9DlcRB2mW759p7XqpKG3wRE-MmPsEPiYTbq6V1Yile4,7826
 nimare/cli.py,sha256=Zvy5jN2KopH_MBke-gm8A0DbBZmIFGvnE1tjhbYib9I,3695
 nimare/correct.py,sha256=2eI0jR6_odND-2CzSlaxRU2um6ccLSapd7ERAYteBnE,17110
@@ -9,7 +9,7 @@ nimare/dataset.py,sha256=pSEccmbqQCYiwfjoy45tgYRTBt1CEFqmKFz_T8p95cA,24814
 nimare/diagnostics.py,sha256=sCatkXUM9rvrY9MMz6Q66njWSiWGifWwSNEMu3ldnVs,20263
 nimare/estimator.py,sha256=DtsSIyPDrKkpi-KNv2m-cZMvJO7CCfHLXHRhCT73sbY,5063
 nimare/generate.py,sha256=L4_c2sLAlF7XDKTm-3q4oOx8pLID2NaxG9YET5KSIZw,12475
-nimare/io.py,sha256=76T9KLOorquImy-01cLG3pwuJLJm_UD_hAhK3_0qj7g,27299
+nimare/io.py,sha256=OKB9qYOto0NSeWHdz5VRBQwO_9aLGzIT6OeNUPwDeLA,28235
 nimare/nimads.py,sha256=2s5QnaLvrBt-kMrImGhG_p6r0unysufIIcPczr2bG0c,24342
 nimare/results.py,sha256=7szcyR6PkZAXBqbIGGWrw1nW9j9QCdpdl4MxUK_1Wzc,8190
 nimare/stats.py,sha256=XhXfFj6KHTPVSTXhbEid0qt8HLqJD82Bl5T23qmaf40,10098
@@ -74,7 +74,7 @@ nimare/tests/test_diagnostics.py,sha256=VrfR_8nQKn2VF7dFdnTM7ZQy3Ou5eHdpaLhml5T6
 nimare/tests/test_estimator_performance.py,sha256=tbK2Qr83rB0in-pB6MccnjLg4iHSyfilx-hTNDWQfe4,12749
 nimare/tests/test_extract.py,sha256=nPaL07G9paLRCJzPOv79jH3mhOPs2YvQdghoLfcDz5A,2348
 nimare/tests/test_generate.py,sha256=LSh2APJsg87u2s2zydkrre3RVk_ZGpoB4d7uuvIPWYE,7318
-nimare/tests/test_io.py,sha256=QKr_zRGu8tyrpiLoLAjCV9ektxCTHRlKPWgyJRqQ9T8,10397
+nimare/tests/test_io.py,sha256=a4n9kMHx2wPIHsQZ1OoMkQq4y5dbhLqecUX3HOOzua8,13618
 nimare/tests/test_meta_ale.py,sha256=hccXSNzLGUgj6E4tCsiHZpuUFoBxXkP293-vtUS5jdE,11791
 nimare/tests/test_meta_cbmr.py,sha256=cl_pUA1dxXpDD5Ci_tllSVG0uKykuneHDbUxGY4w7Ks,9776
 nimare/tests/test_meta_ibma.py,sha256=Yw4F0_pr3cpVSe7oeMlK0-btg1Uw58cenklOsIw87Pc,7775
@@ -111,9 +111,9 @@ nimare/workflows/cbma.py,sha256=2jYJs9kH7_LzFP6d7-oTHiTTgAFbtmiBNtBXSCSZPjg,7052
 nimare/workflows/ibma.py,sha256=lAkWtqSqnZiUUV460Bh046U9LeGhnry3bl8BFi-tx7s,4289
 nimare/workflows/macm.py,sha256=mVUBeKbTawhU93ApnkunZSUXZWo7qBPrM3dMGWfl0ik,2531
 nimare/workflows/misc.py,sha256=OWgHlSAnRI0-5Seii-bd48piIYsfEAF_aNKGorH1yJQ,1827
-nimare-0.5.3rc1.dist-info/LICENSE,sha256=PWPXnCGWh-FMiBZ61OnQ2BHFjPPlJJ7F0kFx_ryzp-M,1074
-nimare-0.5.3rc1.dist-info/METADATA,sha256=ya3k7tn7Hqy8EiKhtPL5uDhoKRiUCHVL5SVFCdhOOpU,4706
-nimare-0.5.3rc1.dist-info/WHEEL,sha256=tZoeGjtWxWRfdplE7E3d45VPlLNQnvbKiYnx7gwAy8A,92
-nimare-0.5.3rc1.dist-info/entry_points.txt,sha256=3w_hk9N2PWnKZkCaJyDlc0_kdn3rh35aiI21rSdvsuA,44
-nimare-0.5.3rc1.dist-info/top_level.txt,sha256=XnOcEXMs0BxdI8t3_ksTl96T8hykn9L7-bxLLraVrTI,18
-nimare-0.5.3rc1.dist-info/RECORD,,
+nimare-0.5.4.dist-info/LICENSE,sha256=PWPXnCGWh-FMiBZ61OnQ2BHFjPPlJJ7F0kFx_ryzp-M,1074
+nimare-0.5.4.dist-info/METADATA,sha256=UPCo4QTIB2g_H8F6hQ2PALnnHPz7eptPhbx5ovfHzZo,4703
+nimare-0.5.4.dist-info/WHEEL,sha256=tZoeGjtWxWRfdplE7E3d45VPlLNQnvbKiYnx7gwAy8A,92
+nimare-0.5.4.dist-info/entry_points.txt,sha256=3w_hk9N2PWnKZkCaJyDlc0_kdn3rh35aiI21rSdvsuA,44
+nimare-0.5.4.dist-info/top_level.txt,sha256=XnOcEXMs0BxdI8t3_ksTl96T8hykn9L7-bxLLraVrTI,18
+nimare-0.5.4.dist-info/RECORD,,