Skip to content

Promote experimental::dynamic_map to cuco::dynamic_map#794

Merged
PointKernel merged 9 commits intoNVIDIA:devfrom
PointKernel:migrate-dynamic-map
Feb 6, 2026
Merged

Promote experimental::dynamic_map to cuco::dynamic_map#794
PointKernel merged 9 commits intoNVIDIA:devfrom
PointKernel:migrate-dynamic-map

Conversation

@PointKernel
Copy link
Copy Markdown
Member

@PointKernel PointKernel commented Feb 2, 2026

This PR promotes cuco::experimental::dynamic_map to cuco::dynamic_map and removes the legacy implementation.

Changes:

  • Added erase, find, contains, retrieve_all methods
  • Added size(), capacity(), load_factor() getters
  • Added sentinel and hash function getters (empty_key_sentinel(), erased_key_sentinel(), key_eq(), hash_function())
  • Updated all tests and benchmarks to use the new API
  • Added insert_or_assign

Closes #732

@PointKernel PointKernel added type: feature request New feature request topic: dynamic_map Issue related to the dynamic_map labels Feb 2, 2026
@PointKernel
Copy link
Copy Markdown
Member Author

@sleeepyjack should we remove legacy dynamic_map directly in this PR so we can start deprecating the legacy static_map next (which is used by the legacy dynamic_map)?

@PointKernel PointKernel added the Needs Review Awaiting reviews before merging label Feb 3, 2026
@PointKernel PointKernel merged commit 8333a0b into NVIDIA:dev Feb 6, 2026
29 checks passed
@PointKernel PointKernel deleted the migrate-dynamic-map branch February 6, 2026 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review Awaiting reviews before merging topic: dynamic_map Issue related to the dynamic_map type: feature request New feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ENHANCEMENT]: dynamic_map::insert_or_assign

2 participants