-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
BUG: spatial.cKDTree: do not slide the median #20606
Open
sturlamolden
wants to merge
10
commits into
scipy:main
Choose a base branch
from
sturlamolden:Fix-cKDTree-sliding-midpoint-error
base: main
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
BUG: spatial.cKDTree: do not slide the median #20606
sturlamolden
wants to merge
10
commits into
scipy:main
from
sturlamolden:Fix-cKDTree-sliding-midpoint-error
Commits on Apr 29, 2024
-
fix sliding midpoint problem in build.cxx
Never slide the pivot when using median as pivot. Only slide the pivot when pivot is the midpoint. That’s what sliding midpoint means.
Configuration menu - View commit details
-
Copy full SHA for 6ea6a4f - Browse repository at this point
Copy the full SHA 6ea6a4fView commit details
Commits on May 8, 2024
-
Configuration menu - View commit details
-
Copy full SHA for b16f6ed - Browse repository at this point
Copy the full SHA b16f6edView commit details -
Configuration menu - View commit details
-
Copy full SHA for ea4f65e - Browse repository at this point
Copy the full SHA ea4f65eView commit details -
Configuration menu - View commit details
-
Copy full SHA for c0f38eb - Browse repository at this point
Copy the full SHA c0f38ebView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6ff4cc7 - Browse repository at this point
Copy the full SHA 6ff4cc7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5b30afe - Browse repository at this point
Copy the full SHA 5b30afeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7bc7253 - Browse repository at this point
Copy the full SHA 7bc7253View commit details -
Configuration menu - View commit details
-
Copy full SHA for b2d2ef7 - Browse repository at this point
Copy the full SHA b2d2ef7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 45dc44d - Browse repository at this point
Copy the full SHA 45dc44dView commit details -
Use sliding rule also on median pivot for stability
Median pivot (aka "standard split") results in stability issues when we have a lot of ties.
Configuration menu - View commit details
-
Copy full SHA for 2304fe7 - Browse repository at this point
Copy the full SHA 2304fe7View commit details
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.