Merge pull request #846 from tsalo/fix-correlationplot

effigies · web-flow · commit 8906e8c01bf2 · 2023-12-11T15:59:42.000-05:00
Correctly use `ignore_initial_volumes` in `ConfoundsCorrelationPlot`
diff --git a/niworkflows/viz/plots.py b/niworkflows/viz/plots.py
@@ -910,7 +910,8 @@ def confounds_correlation_plot(
         confounds_data = confounds_data[list(columns)]
 
     confounds_data = confounds_data.loc[
-        :, np.logical_not(np.isclose(confounds_data.var(skipna=True), 0))
+        ignore_initial_volumes:,
+        np.logical_not(np.isclose(confounds_data.var(skipna=True), 0)),
     ]
     corr = confounds_data.corr()
 

Original file line number	Diff line number	Diff line change
`@@ -910,7 +910,8 @@ def confounds_correlation_plot(`
`910`	`910`	`confounds_data = confounds_data[list(columns)]`
`911`	`911`
`912`	`912`	`confounds_data = confounds_data.loc[`
`913`		`- :, np.logical_not(np.isclose(confounds_data.var(skipna=True), 0))`
	`913`	`+ ignore_initial_volumes:,`
	`914`	`+ np.logical_not(np.isclose(confounds_data.var(skipna=True), 0)),`
`914`	`915`	`]`
`915`	`916`	`corr = confounds_data.corr()`
`916`	`917`