Test Azure on only one version of Python on windows. #685
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1) On appveyor, ABS store represent close to 25% of the test time 8min total, see 10 slowest tests below). 2) Appveyor is the slowest of the CI, the one we wait for most of the time. 3) ABS has not reason to behave differently on different Python versions. 4) We might be able to also test windows on WSL in GH-actions. ``` ========================== slowest 10 test durations ========================== 43.32s call zarr/tests/test_core.py::TestArrayWithABSStore::test_resize_2d 21.28s call zarr/tests/test_hierarchy.py::TestGroupWithABSStore::test_getitem_contains_iterators 17.72s call zarr/tests/test_core.py::TestArrayWithABSStore::test_np_ufuncs 17.54s call zarr/tests/test_core.py::TestArrayWithABSStore::test_array_2d 14.79s call zarr/tests/test_core.py::TestArrayWithABSStore::test_iter 13.76s call zarr/tests/test_core.py::TestArrayWithABSStore::test_append_2d 13.55s call zarr/tests/test_core.py::TestArrayWithABSStore::test_append_2d_axis 12.01s call zarr/tests/test_core.py::TestArrayWithABSStore::test_array_order 11.77s call zarr/tests/test_indexing.py::test_set_orthogonal_selection_3d 8.52s call zarr/tests/test_storage.py::TestFSStore::test_s3_complex ==== 1808 passed, 303 skipped, 3 xpassed, 13 warnings in 333.74s (0:05:33) ==== ```
|
This works and does not decrease coverage, the test time is now down to 4min on the two lowest version of Python (3 min of which are installing package). So next step would be to speedup installing. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
total, see 10 slowest tests below).
time.
versions.