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

[5/n] separare the github build into two conditions #4066

Closed
wants to merge 21 commits into from

Conversation

junjieqi
Copy link
Contributor

@junjieqi junjieqi commented Dec 5, 2024

No description provided.

junjieqi and others added 20 commits December 5, 2024 13:37
Summary:
Pull Request resolved: #4054

Part 1 of more HNSW unit tests

Reviewed By: junjieqi

Differential Revision: D66690398

fbshipit-source-id: 29b35f1c0a919c168fd4cb770552f41eaed5b6b6
Summary:
Pull Request resolved: #4064

Apply matthijs' comments on better naming in D52582914

Reviewed By: junjieqi

Differential Revision: D66822957

fbshipit-source-id: d899dbe89ebd716a18901c734c65dac428856954
Summary:
Pull Request resolved: #4056

Part 2 of more HNSW unit tests

Added comments indicating some currently unused code.

Reviewed By: junjieqi

Differential Revision: D66782376

fbshipit-source-id: 5a7c210d516424e58054f36f0a74249859e48a8f
Summary:
Pull Request resolved: #4065

improve neighbors_2 naming from codemod

Reviewed By: junjieqi

Differential Revision: D66823395

fbshipit-source-id: 90af06e7f6d917581a25311944689f82f12c93e1
@facebook-github-bot
Copy link
Contributor

@junjieqi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@junjieqi merged this pull request in 0d568bc.

@junjieqi junjieqi deleted the separate_workflow branch December 6, 2024 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants