-
Notifications
You must be signed in to change notification settings - Fork 69
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
MAINT: Test result caching #836
Conversation
* upstream/main: FIX: Paths and pyarrow (mne-tools#834)
@hoechenberger this appears headed for green. I was unable to replicate the caching issue on macOS, both datasets tested here appear to process in < 15s the second time. Ready for review/merge from my end. Happy to adjust CI requirements if you're happy (I folded the Style check into the doc consistency one since the only step we really needed there now was |
Thanks! I also realized I cannot reliably reproduce on macOS. Sometimes the problem occurs and sometimes it doesn't. I usually work inside of a OneDrive folder, maybe this has implications... |
It seems like it could for |
Let's just go ahead here and I'll report back sometime in the future if I run into this issue again :) |
Before merging …
docs/source/changes.md
)Closes #814
Let's see if we can replicate the issue! EDIT: Spoiler alert, we cannot.