Merge pull request #104 from eurunuela/feat/use-non-zscore-ica-maps

[FIX] Use non-z-scored ICA component maps

Author: eurunuela

features.json

@@ -288,7 +288,7 @@
       "category": "visualization",
       "description": "Display interactive brain stat map mosaic using Niivue for selected ICA component",
       "steps": [
-        "Load a tedana folder containing *stat-z_components.nii.gz",
+        "Load a tedana folder containing *_components.nii.gz",
         "Navigate to the 'ICA' tab",
         "Verify brain viewer displays 3 rows of slices (axial, sagittal, coronal) with 7 slices per row",
         "Verify warm/cool diverging colormap is applied (warm for positive, cool for negative)",

scripts/rica_server.py

@@ -26,7 +26,7 @@
 RICA_FILE_PATTERNS = [
     "_metrics.tsv",
     "_mixing.tsv",
-    "stat-z_components.nii.gz",
+    "_components.nii.gz",
     "_mask.nii",
     "report.txt",
     "comp_",
@@ -87,7 +87,9 @@ def send_file_list(self):
         for f in cwd.rglob("*"):
             if f.is_file():
                 # Check if file matches any Rica pattern
-                if any(pattern in f.name for pattern in RICA_FILE_PATTERNS):
+                if any(pattern in f.name for pattern in RICA_FILE_PATTERNS) and not (
+                    "_components.nii.gz" in f.name and "stat-z" in f.name
+                ):
                     # Store relative path with forward slashes
                     rel_path = str(f.relative_to(cwd)).replace("\\", "/")
                     files.append(rel_path)
@@ -137,7 +139,11 @@ def main():
     cwd = Path.cwd()
     rica_files = []
     for f in cwd.rglob("*"):
-        if f.is_file() and any(p in f.name for p in RICA_FILE_PATTERNS):
+        if (
+            f.is_file()
+            and any(p in f.name for p in RICA_FILE_PATTERNS)
+            and not ("_components.nii.gz" in f.name and "stat-z" in f.name)
+        ):
             rica_files.append(f.name)
 
     if not rica_files:

src/PopUps/IntroPopUp.js

@@ -143,7 +143,7 @@ function IntroPopup({ onDataLoad, onLoadingStart, closePopup, isLoading, isDark
           (f.includes("_metrics.tsv") && !f.toLowerCase().includes("pca")) ||
           (f.startsWith("tedana_20") && f.endsWith(".tsv")) ||
           (f.includes("_mixing.tsv") && !f.toLowerCase().includes("pca") && !f.toLowerCase().includes("orth")) ||
-          (f.includes("stat-z_components.nii.gz") && f.toLowerCase().includes("ica")) ||
+          (f.includes("_components.nii.gz") && f.toLowerCase().includes("ica") && !f.includes("stat-z")) ||
           f === "betas_OC.nii.gz" ||
           f.includes("_mask.nii") ||
           f.includes("CrossComponent_metrics.json") ||
@@ -261,8 +261,8 @@ function IntroPopup({ onDataLoad, onLoadingStart, closePopup, isLoading, isDark
             setLoadingProgress((prev) => ({ ...prev, current: prev.current + 1 }));
           }
 
-          // ICA stat-z components NIfTI
-          if ((filename.includes("stat-z_components.nii.gz") && filename.toLowerCase().includes("ica")) || filename === "betas_OC.nii.gz") {
+          // ICA components NIfTI
+          if ((filename.includes("_components.nii.gz") && filename.toLowerCase().includes("ica") && !filename.includes("stat-z")) || filename === "betas_OC.nii.gz") {
             const response = await fetch(`/${filepath}`);
             niftiBuffer = await response.arrayBuffer();
             setLoadingProgress((prev) => ({ ...prev, current: prev.current + 1 }));
@@ -408,7 +408,7 @@ function IntroPopup({ onDataLoad, onLoadingStart, closePopup, isLoading, isDark
           (f.name.startsWith("tedana_20") && f.name.endsWith(".tsv")) ||
           // New files for Niivue integration
           (f.name.includes("_mixing.tsv") && !f.name.toLowerCase().includes("pca") && !f.name.toLowerCase().includes("orth")) ||
-          (f.name.includes("stat-z_components.nii.gz") && f.name.toLowerCase().includes("ica")) ||
+          (f.name.includes("_components.nii.gz") && f.name.toLowerCase().includes("ica") && !f.name.includes("stat-z")) ||
           f.name === "betas_OC.nii.gz" ||
           f.name.includes("_mask.nii") ||
           f.name.includes("CrossComponent_metrics.json") ||
@@ -519,8 +519,8 @@ function IntroPopup({ onDataLoad, onLoadingStart, closePopup, isLoading, isDark
             setLoadingProgress((prev) => ({ ...prev, current: prev.current + 1 }));
           }
 
-          // ICA stat-z components NIfTI (4D brain maps for Niivue)
-          if ((filename.includes("stat-z_components.nii.gz") && filename.toLowerCase().includes("ica")) || filename === "betas_OC.nii.gz") {
+          // ICA components NIfTI (4D brain maps for Niivue)
+          if ((filename.includes("_components.nii.gz") && filename.toLowerCase().includes("ica") && !filename.includes("stat-z")) || filename === "betas_OC.nii.gz") {
             niftiBuffer = await readFileAsArrayBuffer(file);
             setLoadingProgress((prev) => ({ ...prev, current: prev.current + 1 }));
           }