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.

Sign up for GitHub

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

Jump to bottom

Add pair_contains in single map #191

Closed
wants to merge 3 commits into from
Closed

Add pair_contains in single map #191

wants to merge 3 commits into from

Conversation

PointKernel
Copy link
Member

Closes #173

This PR adds static_map::pair_contains which is a dependency for fixing rapidsai/cudf#11299.

@PointKernel PointKernel added type: feature request New feature request topic: static_map Issue related to the static_map helps: rapids Helps or needed by RAPIDS In Progress Currently a work in progress labels Jul 20, 2022
This was referenced Jul 20, 2022
Closed
Closed
@PointKernel
Copy link
Member Author

rerun tests

@PointKernel
Copy link
Member Author

Closing this issue since the functionality will be superseded by static_set::contains.

@PointKernel PointKernel deleted the static_map_pair_contains branch December 8, 2022 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sleeepyjack sleeepyjack Awaiting requested review from sleeepyjack sleeepyjack will be requested when the pull request is marked ready for review sleeepyjack is a code owner

Assignees
No one assigned
Labels
helps: rapids Helps or needed by RAPIDS In Progress Currently a work in progress topic: static_map Issue related to the static_map type: feature request New feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEA] Support pair_contains in static_map and static_multimap
1 participant
@PointKernel