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 costly redundant ElasticEnsemble classifier doctest #4981

Merged
merged 1 commit into from Aug 4, 2023
Merged

[MNT] speed up costly redundant ElasticEnsemble classifier doctest #4981

merged 1 commit into from Aug 4, 2023

Conversation

fkiraly
Copy link
Collaborator

@fkiraly fkiraly commented Jul 29, 2023

This PR speeds up tests by skipping the costly, redundant ElasticEnsemble classifier doctest.

The doctest is redundant with the basic fit/predict interface contract test in TestAllClassifiers, but costs 10-20s of test time.

@fkiraly fkiraly added maintenance Continuous integration, unit testing & package distribution module:tests test framework functionality - only framework, excl specific tests labels Jul 29, 2023
@fkiraly fkiraly merged commit 20a7631 into main Aug 4, 2023
24 checks passed
@fkiraly fkiraly deleted the skip-ElasticEnsemble-doctest branch August 4, 2023 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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