Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Allow segmented problems to have different types for offset iterators. #291

Conversation

psvvsp
Copy link
Contributor

@psvvsp psvvsp commented Apr 28, 2021

Modified segmented reduce, segmented radix sort and their tests.

Modified segmented reduce, segmented radix sort and their tests.
Copy link
Collaborator

@alliepiper alliepiper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the patch, this looks great!

One small change needed in the tests -- the transform iterator header and implementation are mismatched. Once that's fixed I'll kick off testing for this.

test/test_device_radix_sort.cu Outdated Show resolved Hide resolved
test/test_device_reduce.cu Outdated Show resolved Hide resolved
@alliepiper alliepiper linked an issue May 4, 2021 that may be closed by this pull request
@alliepiper alliepiper added this to the 1.13.0 milestone May 4, 2021
@alliepiper alliepiper assigned alliepiper and unassigned psvvsp May 5, 2021
@alliepiper
Copy link
Collaborator

Looks good -- thanks again for this! I know these large, tedious refactorings aren't fun, and I appreciate your effort here :)

I'll kick off testing today, we should be able to get this into the next release.

alliepiper added a commit to alliepiper/thrust that referenced this pull request May 5, 2021
@alliepiper
Copy link
Collaborator

alliepiper commented May 5, 2021

gpuCI: NVIDIA/thrust#1422
DVS: 29929849

@alliepiper alliepiper added testing: internal ci in progress Currently testing on internal NVIDIA CI (DVS). testing: gpuCI in progress Started gpuCI testing. labels May 5, 2021
@psvvsp
Copy link
Contributor Author

psvvsp commented May 5, 2021

Will you write me if the tests fail? I'm ready to fix bugs if you provide me with build configuration.

@alliepiper
Copy link
Collaborator

Sure! I'll let you know how it goes.

@alliepiper
Copy link
Collaborator

@psvvsp Tests passed, this is all set! I'll merge this PR during the next round of integrations.

@alliepiper alliepiper added testing: internal ci passed Passed internal NVIDIA CI (DVS). testing: gpuCI passed Passed gpuCI testing. and removed testing: internal ci in progress Currently testing on internal NVIDIA CI (DVS). testing: gpuCI in progress Started gpuCI testing. labels May 10, 2021
@alliepiper alliepiper merged commit 0554d8f into NVIDIA:main May 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
testing: gpuCI passed Passed gpuCI testing. testing: internal ci passed Passed internal NVIDIA CI (DVS).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow segmented problems to have different types for offset iterator
2 participants