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.

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

Transform euclidean #11

Merged
merged 26 commits into from
Nov 24, 2023
Merged

Transform euclidean #11

merged 26 commits into from
Nov 24, 2023

Conversation

rchan26
Copy link
Collaborator

@rchan26 rchan26 commented Oct 24, 2023

Implementation of first transforming the space and using Euclidean distance. Must post-process the distances and set to either infinity or zero depending on some thresholds.

@rchan26 rchan26 requested a review from jh83775 November 24, 2023 14:43
Copy link
Contributor

@jh83775 jh83775 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Ryan, thanks for this - looks good to me!

@rchan26 rchan26 merged commit ee766b8 into pynndescent Nov 24, 2023
15 checks passed
@rchan26 rchan26 deleted the transform-euclidean branch November 24, 2023 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants