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.

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

Provide device_type enum #985

Closed
wants to merge 9 commits into from

Conversation

wphicks
Copy link
Contributor

@wphicks wphicks commented Nov 3, 2022

As a follow-up to #984 which introduces a memory_type enum, this PR introduces a device_type enum with similar justifications. Here, there is less concern about invalid state, but it is still useful to separate the concept of a compute device type from an execution policy. This also allows for future extensibility.

wphicks and others added 5 commits November 3, 2022 14:28
Provide an enum class used to distinguish among various memory types.
This allows decoupling of the concept of memory type from a specific
mdspan accessor policy and ensures that templates receive only a
logically-consistent memory type.
As a follow-up to rapidsai#984 which introduces a memory_type enum, this PR
introduces a device_type enum with similar justifications.
@wphicks wphicks requested review from a team as code owners November 3, 2022 19:12
@wphicks wphicks changed the title Provide device type enum Provide device_type enum Nov 3, 2022
@wphicks wphicks added feature request New feature or request non-breaking Non-breaking change labels Nov 3, 2022
@wphicks wphicks requested review from a team as code owners March 6, 2023 18:53
@wphicks wphicks changed the base branch from branch-22.12 to branch-23.04 March 6, 2023 18:55
@wphicks wphicks closed this Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

1 participant