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

Tpetra: Purge deprecated Kokkos functions ptr_on_device and dimension_N #2705

Closed
mhoemmen opened this issue May 9, 2018 · 2 comments
Closed

Comments

@mhoemmen
Copy link
Contributor

mhoemmen commented May 9, 2018

Kokkos has already deprecated and will soon remove Kokkos::View methods ptr_on_device and dimension_N (replace N with 0, 1, 2, etc.). Sierra (@rrdrake , @prwolfe ) is already seeing build errors, since they don't enable the macro that enables deprecated code. I will fix this in Tpetra. Replace ptr_on_device with data and dimension_N with extent(N).

@trilinos/tpetra

@mhoemmen mhoemmen self-assigned this May 9, 2018
mhoemmen pushed a commit that referenced this issue May 9, 2018
@trilinos/tpetra

Kokkos::View methods ptr_on_device() and dimension_N() (replace N with
0, 1, 2, ...) have been deprecated for a long time.  Kokkos has
replaced them with data() resp. extent(N).

This commit replaces all uses of the deprecated methods in Tpetra,
with their nondeprecated replacements.  This fixes #2705.
@mhoemmen
Copy link
Contributor Author

mhoemmen commented May 9, 2018

PR #2707 has a fix for this issue.

mhoemmen pushed a commit that referenced this issue May 9, 2018
@trilinos/tpetra

Kokkos::View methods ptr_on_device() and dimension_N() (replace N with
0, 1, 2, ...) have been deprecated for a long time.  Kokkos has
replaced them with data() resp. extent(N).

This commit replaces all uses of the deprecated methods in Tpetra,
with their nondeprecated replacements.  This fixes #2705.
@mhoemmen
Copy link
Contributor Author

mhoemmen commented May 9, 2018

I just merged PR #2707 into develop, that fixes this issue. Thanks all!

@mhoemmen mhoemmen closed this as completed May 9, 2018
mhoemmen pushed a commit that referenced this issue May 9, 2018
mhoemmen pushed a commit that referenced this issue May 9, 2018
@trilinos/xpetra @trilinos/muelu See #2705 for context.
mhoemmen pushed a commit that referenced this issue May 9, 2018
@trilinos/trilinoscouplings See #2705 for context.
mhoemmen pushed a commit that referenced this issue May 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant