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.

Sign up for GitHub

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

Jump to bottom

[MNT] speed up various non-suite tests, part 2 #5071

Merged
merged 11 commits into from Aug 13, 2023
Merged

[MNT] speed up various non-suite tests, part 2 #5071

merged 11 commits into from Aug 13, 2023

Conversation

fkiraly
Copy link
Collaborator

@fkiraly fkiraly commented Aug 10, 2023 •
edited

This PR speeds up various non-suite tests:

  • test_gscv_hierarchical, by shortening the hierarchical data used in the test
  • tests for classifiers in test_sklearn_compatibility, by using a smaller data set
  • test_docs_tsfresh_extractor, by using a smaller data set

@fkiraly fkiraly added maintenance Continuous integration, unit testing & package distribution module:tests test framework functionality - only framework, excl specific tests labels Aug 10, 2023
@fkiraly
Copy link
Collaborator Author

fkiraly commented Aug 13, 2023

merging early - to make iterations for release earlier (this speeds up the tests substantially)

@fkiraly fkiraly merged commit 1d299b4 into main Aug 13, 2023
24 checks passed
@fkiraly fkiraly deleted the speed-tests-various2 branch August 13, 2023 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@achieveordie achieveordie Awaiting requested review from achieveordie achieveordie is a code owner

@benHeid benHeid Awaiting requested review from benHeid benHeid is a code owner

@yarnabrina yarnabrina Awaiting requested review from yarnabrina yarnabrina is a code owner

Assignees
No one assigned
Labels
maintenance Continuous integration, unit testing & package distribution module:tests test framework functionality - only framework, excl specific tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@fkiraly