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

MRG, MAINT: Speed up tests #8665

Merged
merged 8 commits into from
Dec 17, 2020
Merged

MRG, MAINT: Speed up tests #8665

merged 8 commits into from
Dec 17, 2020

Conversation

larsoner
Copy link
Member

@larsoner larsoner commented Dec 16, 2020

Looking at some of the slowest ones on the Azure test that keeps failing (avoiding 3D viz tests for now) on master I get these timings:

75.15s total   mne/preprocessing/tests/
57.95s total   mne/beamformer/tests/
28.77s total   mne/io/fiff/
21.70s total   mne/io/fieldtrip/
22.52s total   mne/io/egi/
14.10s total   mne/io/eeglab/

On this PR, which mostly uses crop and reduction of redundant test cases to speed things up, I get:

55.78s total   mne/preprocessing/tests/
47.89s total   mne/beamformer/tests/
24.87s total   mne/io/fiff/
13.01s total   mne/io/fieldtrip/
 7.62s total   mne/io/egi/
 9.59s total   mne/io/eeglab/

So at least some incremental progress locally, and hopefully differences will be even better on CIs.

@larsoner larsoner added this to the 0.22 milestone Dec 16, 2020
@agramfort agramfort merged commit 1b9d222 into mne-tools:master Dec 17, 2020
@agramfort
Copy link
Member

thx @larsoner

@larsoner larsoner deleted the test-time branch December 17, 2020 12:04
@larsoner larsoner mentioned this pull request Jun 3, 2022
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

Successfully merging this pull request may close these issues.

2 participants