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

ENH: Add additional plots and customization options to interative mode #1004

Open
scholzri opened this issue Oct 10, 2024 · 1 comment
Open

Comments

@scholzri
Copy link

Currently, when using mne-bids-pipeline with interactive=True, there are several key plots that would be valuable to include for more comprehensive data inspection and quality control. I propose the following enhancements:

  1. Add raw.plot() before preprocessing/_01_data_quality:
  • Allow for checking and marking bad channels early in the pipeline.
  1. Include ica.plot_components(), ica.plot_sources(), and ica.plot_scores() after preprocessing/_06a2_find_ica_artifacts:
  • Enable review and modification of automatic ICA selection if necessary.
  1. Add epochs.plot() after preprocessing/_09_ptp_reject:
  • Facilitate visualization and manual rejection of epochs.

Additionally, it would be beneficial to allow users to specify which plots they want to see in interactive mode. For example:

interactive = ['raw', 'ica', 'epochs']

This customization would give users more control over the interactive process while maintaining efficiency for those who don't need all plots.

@larsoner
Copy link
Member

Personally I don't use the diagnostic plots and just inspect reports so I don't have a lot of first-hand insight here. But what you suggest adding seems to make sense, both in terms of additional plots and in terms of how to subselect different interactive sections/levels! So I would say yes if you're up for implementing these a PR would be great.

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