-
Notifications
You must be signed in to change notification settings - Fork 20.5k
Quickselect median of medians #6661
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
base: master
Are you sure you want to change the base?
Quickselect median of medians #6661
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6661 +/- ##
============================================
+ Coverage 75.75% 75.79% +0.04%
- Complexity 5772 5791 +19
============================================
Files 703 704 +1
Lines 19777 19821 +44
Branches 3832 3842 +10
============================================
+ Hits 14982 15024 +42
- Misses 4215 4216 +1
- Partials 580 581 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Updated imports in DeterministicQuickSelectTest.java to explicitly list JUnit methods, resolving the 'AvoidStarImport' Checkstyle rule. Also re-added the missing 'org.junit.jupiter.api.Test' import.
Adjusted the import order in DeterministicQuickSelectTest.java, placing regular imports before static imports, to satisfy the project's formatting and linting rules.
Removed trailing comments on import statements and ensured correct ordering (regular before static) in DeterministicQuickSelectTest.java to pass the project's formatting checks.
@lukmanu1 What's the difference between your implementation and the existing |
This PR adds a deterministic QuickSelect algorithm using the Median of Medians method in the divideandconquer package.
Closes #6660
clang-format -i --style=file path/to/your/file.java