Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Overhaul of the carpet plots #665

Open
oesteban opened this issue Nov 4, 2021 · 1 comment
Open

Overhaul of the carpet plots #665

oesteban opened this issue Nov 4, 2021 · 1 comment

Comments

@oesteban
Copy link
Member

oesteban commented Nov 4, 2021

We need several improvements to be done to the carpet plots:

  • Improve the readability (and perhaps a colorblind-safe color scale) of the indication of segments (WM, cortical GM, CSF, etc.), by making the indicator thicker, perhaps adding a minimal separation between segments, etc. Alternatively, add textual legends
  • Better labeling of axes
  • Ward clustering
  • Reconsider the standardization strategies (perhaps all data standardized the same way, rather than region-wise?)
  • Ensure we are dropping nonsteady states (I think we just merged that with ENH: carpet plot tweaks #617).
  • Tweaks to the timeseries represented on top:
    • Global signals, FD+DVARS, CompCor(s) can be plotted sharing x-axis.
    • Perhaps plot a kernel density estimation on the right showing the distribution of confound timeseries.
  • Chris' idea of delineating CompCor ROIs on the carpet.
@effigies
Copy link
Member

effigies commented Nov 4, 2021

For colorblindness, I would default to Seaborn's colorblind palette: https://seaborn.pydata.org/tutorial/color_palettes.html#qualitative-color-palettes

And here's a gist with simulated contrasts under different colorblindness conditions: https://gist.github.com/mwaskom/b35f6ebc2d4b340b4f64a4e28e778486

I believe we need 7 colors if we move to CIFTI and splitting the aCompCor mask out of WM+CSF: crown, lh, rh, subcortical, cerebellum, WM+CSF-aCompCor, aCompCor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants