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

BUG: Parallelization over runs randomizes report order #845

Open
larsoner opened this issue Feb 6, 2024 · 0 comments
Open

BUG: Parallelization over runs randomizes report order #845

larsoner opened this issue Feb 6, 2024 · 0 comments

Comments

@larsoner
Copy link
Member

larsoner commented Feb 6, 2024

Currently we parallelize over subjects and runs. This leads to stuff where in parallel mode run 4 can be processed faster than run 1, and thus appears in report first. In serial mode this won't happen and run order is preserved. I've opened mne-tools/mne-python#12424 to tackle this, but once that (or something similar) lands, we should also reconsider ordering more generally. Take for example this TOC:

It would be better to ravel the sections differently, namely have all run 1 parts first, then all run 2 parts second, etc. Typically a user wants to compare these bits sequentially, and I find it annoying to have to scroll super far to do it.

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

1 participant