Skip to content

Comments

[cuda.compute]: Fixes and updates to benchmarks#6999

Merged
shwina merged 3 commits intoNVIDIA:mainfrom
shwina:use-central-cache-and-update-benchmarks
Dec 19, 2025
Merged

[cuda.compute]: Fixes and updates to benchmarks#6999
shwina merged 3 commits intoNVIDIA:mainfrom
shwina:use-central-cache-and-update-benchmarks

Conversation

@shwina
Copy link
Contributor

@shwina shwina commented Dec 17, 2025

Description

This PR updates our benchmarks to work again. They were broken in #6938, due to the indirection introduced in the per-algorithm cache.

I've now moved to a central cache rather than a per-algorithm cache, and introduced the convenience function cuda.compute.clear_all_caches().

Also added a new bench_select.py.

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@shwina shwina requested review from a team as code owners December 17, 2025 16:25
@shwina shwina requested a review from NaderAlAwar December 17, 2025 16:25
@github-project-automation github-project-automation bot moved this to Todo in CCCL Dec 17, 2025
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Dec 17, 2025
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@shwina shwina force-pushed the use-central-cache-and-update-benchmarks branch from 5880ad5 to ef42148 Compare December 18, 2025 18:00
@github-actions

This comment has been minimized.

@shwina shwina force-pushed the use-central-cache-and-update-benchmarks branch from ef42148 to 587939e Compare December 19, 2025 09:58
@shwina shwina enabled auto-merge (squash) December 19, 2025 09:59
@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

🥳 CI Workflow Results

🟩 Finished in 3h 30m: Pass: 100%/48 | Total: 13h 42m | Max: 42m 00s

See results here.

@shwina shwina merged commit 48999a3 into NVIDIA:main Dec 19, 2025
118 of 123 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Dec 19, 2025
fbusato pushed a commit to fbusato/cccl that referenced this pull request Dec 23, 2025
* Move algorithm cache to a central registry

* Update select benchmark

* Update merge_sort benchmark

---------

Co-authored-by: Ashwin Srinath <shwina@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants