lina-usc / pylossless

🧠 EEG Processing pipeline that annotates continuous data
https://pylossless.readthedocs.io/en/latest/
MIT License
18 stars 8 forks source link

Move slider sharing duties to `qcgui` #62

Closed scott-huberty closed 1 year ago

scott-huberty commented 1 year ago

Fixes #61

  1. This removes some of the complexity out of mne_visualizer, as it is simpler to handle slider sharing directly in our qcr app.

  2. I think this improves the API of MNEVisualizer. no more time_slider ch_slider args that require a dash id. There are new, clear, args named show_ch_slider / show_time_slider, for show showing/hiding sliders.

  3. Now, you can control the topoplot view directly with its own slider. The ICA time-series and ICA-Topo plot sliders are linked. Want to hide it? NP, just pass show_slider=False to the TopoVizICA constructor.

Screen Shot 2023-03-15 at 9 44 25 AM