Skip to content

utility: Add custom pair implementation - #425

Merged
stotko merged 1 commit into
masterfrom
pair
Jul 17, 2024
Merged

utility: Add custom pair implementation#425
stotko merged 1 commit into
masterfrom
pair

Conversation

@stotko

@stotko stotko commented Jul 17, 2024

Copy link
Copy Markdown
Owner

With thrust 2.3.0+ (bundled in CUDA 12.4+), their implementation of pair have been replaced by cuda:std::pair from libcudacxx. Unfortunately, this causes compiler errors due to ambiguous namespace resolution for overlapping functionality between stdgpu and libcudacxx. Replace the alias to thrust's pair class with a custom implementation to fix these issues.

This further reduces the dependency to thrust, see #279.

@stotko stotko added this to the 2.0.0 milestone Jul 17, 2024
@codecov

codecov Bot commented Jul 17, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.38%. Comparing base (1b6a331) to head (6976d5b).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #425      +/-   ##
==========================================
+ Coverage   97.33%   97.38%   +0.05%     
==========================================
  Files          31       32       +1     
  Lines        2512     2524      +12     
==========================================
+ Hits         2445     2458      +13     
+ Misses         67       66       -1     

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

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant