Skip to content

Fix the launch bounds for nn-descent kernel for __CUDA_ARCH__ 1210#965

Closed
achirkin wants to merge 1 commit intobranch-25.08from
achirkin-nn-descent-1210
Closed

Fix the launch bounds for nn-descent kernel for __CUDA_ARCH__ 1210#965
achirkin wants to merge 1 commit intobranch-25.08from
achirkin-nn-descent-1210

Conversation

@achirkin
Copy link
Copy Markdown
Contributor

A fix to make cuVS build for a target architecture 1210

@achirkin achirkin self-assigned this May 30, 2025
@achirkin achirkin requested a review from a team as a code owner May 30, 2025 14:40
@achirkin achirkin added bug Something isn't working non-breaking Introduces a non-breaking change labels May 30, 2025
@github-actions github-actions Bot added the cpp label May 30, 2025
@viclafargue viclafargue requested a review from jinsolp May 30, 2025 14:51
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (branch-25.08@36c38a5). Learn more about missing BASE report.

Additional details and impacted files
@@               Coverage Diff               @@
##             branch-25.08     #965   +/-   ##
===============================================
  Coverage                ?   83.20%           
===============================================
  Files                   ?       21           
  Lines                   ?      131           
  Branches                ?        0           
===============================================
  Hits                    ?      109           
  Misses                  ?       22           
  Partials                ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jakirkham
Copy link
Copy Markdown
Member

Given this is a simple fix, could we target for 25.06?

@achirkin achirkin changed the base branch from branch-25.08 to branch-25.06 June 2, 2025 06:22
@achirkin achirkin requested review from a team as code owners June 2, 2025 06:22
@achirkin achirkin requested a review from AyodeAwe June 2, 2025 06:22
@achirkin achirkin changed the base branch from branch-25.06 to branch-25.08 June 2, 2025 06:23
@achirkin achirkin removed request for a team, AyodeAwe and jinsolp June 2, 2025 06:23
achirkin added a commit that referenced this pull request Jun 2, 2025
@achirkin
Copy link
Copy Markdown
Contributor Author

achirkin commented Jun 2, 2025

closing in favor of #967

@achirkin achirkin closed this Jun 2, 2025
copy-pr-bot Bot pushed a commit that referenced this pull request Jun 3, 2025
Backport of #965
Invert the CUDA_ARCH logic

Fix ==
Fix style
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working cpp non-breaking Introduces a non-breaking change

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants