Skip to content

Fix GPU PQ build errors and added error checks - #334

Merged
Jie (Diego) Cai (caijiediego) merged 9 commits into
microsoft:mainfrom
bkarsin:fix_gpu_pq_build
Sep 27, 2022
Merged

Fix GPU PQ build errors and added error checks#334
Jie (Diego) Cai (caijiediego) merged 9 commits into
microsoft:mainfrom
bkarsin:fix_gpu_pq_build

Conversation

@bkarsin

Copy link
Copy Markdown
Contributor

Several fixes to build issues with GPU PQ, including:

  • Removed cosine distance code from GPU PQ (not supported with PQ),
  • Fixed issue with GPUPQ_Quantizer causing build error,
  • Added checks and error messages for GPU PQ

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Fixed GPU build break on both Linux and windows. Tested locally on windows.

MaggieQi
MaggieQi previously approved these changes Sep 26, 2022

@MaggieQi MaggieQi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

approve

@caijiediego
Jie (Diego) Cai (caijiediego) dismissed stale reviews from MaggieQi and themself via 6242cff September 26, 2022 13:06

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Try to trigger GitHub Test

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approve

@caijiediego
Jie (Diego) Cai (caijiediego) merged commit ca61760 into microsoft:main Sep 27, 2022
L1ray (L1-ray) pushed a commit to L1-ray/SPTAG that referenced this pull request May 9, 2026
Fix GPU PQ build errors and added error checks
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.

4 participants