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

Sacado, Stokhos: Kokkos::Impl::Mirror[DRV]Type have been removed upstream #13517

Closed
dalg24 opened this issue Oct 9, 2024 · 5 comments
Closed
Labels
type: bug The primary issue is a bug in Trilinos code or tests

Comments

@dalg24
Copy link
Contributor

dalg24 commented Oct 9, 2024

Bug Report

See these two bug reports on the Kokkos side:
kokkos/kokkos#7413 and kokkos/kokkos#7415

Following the merge of kokkos/kokkos#7339 where we got rid of the Kokkos::Impl::Mirror[DRV]Type trait classes, our Trilinos nightly builds are failing due to Sacado and Stokhos relying on these private implementation details.

cc @etphipp @ndellingwood

Description

These are the places in Sacado and Stokhos that need to be updated:

Here you'd probably want to change Impl::MirrorType -> Impl::MirrorViewType

packages/sacado/src/KokkosExp_View_Fad.hpp:487:  typename Impl::MirrorType<Space,T,P ...>::view_type>::type
packages/sacado/src/KokkosExp_View_Fad.hpp:493:  return typename Impl::MirrorType<Space,T,P ...>::view_type(src.label(),layout);
packages/sacado/src/KokkosExp_View_Fad.hpp:568:  typename Impl::MirrorType<Space,T,P ...>::view_type>::type
packages/sacado/src/KokkosExp_View_Fad.hpp:575:  return typename Impl::MirrorType<Space,T,P ...>::view_type(
packages/sacado/src/Kokkos_View_Fad_Fwd.hpp:73:struct MirrorType;
packages/sacado/src/Kokkos_View_Fad_Fwd.hpp:113:  typename Impl::MirrorType<Space,T,P ...>::view_type>::type
packages/sacado/src/Kokkos_View_Fad_Fwd.hpp:149:  typename Impl::MirrorType<Space,T,P ...>::view_type>::type
packages/stokhos/src/sacado/kokkos/pce/KokkosExp_View_UQ_PCE_Contiguous.hpp:232:  typename Impl::MirrorType<Space,T,P ...>::view_type>::type
packages/stokhos/src/sacado/kokkos/pce/KokkosExp_View_UQ_PCE_Contiguous.hpp:238:  return typename Impl::MirrorType<Space,T,P ...>::view_type(
packages/stokhos/src/sacado/kokkos/pce/KokkosExp_View_UQ_PCE_Contiguous.hpp:307:  typename Impl::MirrorType<Space,T,P ...>::view_type>::type
packages/stokhos/src/sacado/kokkos/pce/KokkosExp_View_UQ_PCE_Contiguous.hpp:314:  return typename Impl::MirrorType<Space,T,P ...>::view_type(
packages/stokhos/src/sacado/kokkos/pce/Kokkos_View_UQ_PCE_Fwd.hpp:41:    struct MirrorType;
packages/stokhos/src/sacado/kokkos/pce/Kokkos_View_UQ_PCE_Fwd.hpp:88:  typename Impl::MirrorType<Space,T,P ...>::view_type>::type
packages/stokhos/src/sacado/kokkos/pce/Kokkos_View_UQ_PCE_Fwd.hpp:119:  typename Impl::MirrorType<Space,T,P ...>::view_type>::type
packages/stokhos/src/sacado/kokkos/vector/KokkosExp_View_MP_Vector_Contiguous.hpp:131:    return typename Impl::MirrorType<typename Impl::ViewCtorProp<ViewCtorArgs...>::memory_space, T, P ...>::view_type(prop_copy, layout);
packages/stokhos/src/sacado/kokkos/vector/Kokkos_View_MP_Vector_Fwd.hpp:41:    struct MirrorType;

and there for DynRankView you'd want Impl::MirrorDRVType -> Impl::MirrorDRViewType

packages/sacado/src/Kokkos_DynRankView_Fad.hpp:47:struct MirrorDRVType;
packages/sacado/src/Kokkos_DynRankView_Fad.hpp:96:typename Impl::MirrorDRVType<Space,T,P ...>::view_type
packages/sacado/src/Kokkos_DynRankView_Fad.hpp:1249:typename Impl::MirrorDRVType<Space,T,P ...>::view_type
packages/sacado/src/Kokkos_DynRankView_Fad.hpp:1260:  return typename Impl::MirrorDRVType<Space,T,P ...>::view_type(
@dalg24 dalg24 added the type: bug The primary issue is a bug in Trilinos code or tests label Oct 9, 2024
@dalg24 dalg24 changed the title Sacado, Stokhos: General Summary of the Bug Sacado, Stokhos: Kokkos::Impl::Mirror[DRV]Type have been removed upstream Oct 9, 2024
@pzehner
Copy link

pzehner commented Oct 9, 2024

I could build the code by expunging the use of Kokkos::Impl::MirrorType in packages/stokhos:

Kokkos::Impl::MirrorType</* ... */>::view_type

becomes

Kokkos::Impl::MirrorViewType</* ... */>::dest_view_type

@ndellingwood
Copy link
Contributor

@dalg24 @pzehner thanks for helping diagnose, looks like a simple enough update. I'll gather this into a PR and list you as co-authors, unless you already have PR in progress

@dalg24
Copy link
Contributor Author

dalg24 commented Oct 9, 2024

(Per Kokkos developer meeting Nathan will open a PR)

@ndellingwood
Copy link
Contributor

I'm open a PR once my local testing is complete with kokkos@develop

@dalg24
Copy link
Contributor Author

dalg24 commented Oct 10, 2024

Fixed by #13521
Thanks

@dalg24 dalg24 closed this as completed Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug The primary issue is a bug in Trilinos code or tests
Projects
None yet
Development

No branches or pull requests

3 participants