Skip to content

Conversation

@cwsmith
Copy link
Collaborator

@cwsmith cwsmith commented Nov 6, 2025

This PR adds MeshFields as a new TPL that provides a superconvergent patch recovery implementation to compute a size field to drive mesh adaptation. A 2d adaptive version of the 2d transient stk test tests/corePDEs/AdvDiff/input_2d_stk.yaml was added to exercise the functionality.

CMake flags to enable UVM support in Omega_h is included in this PR and was tested on Perlmutter GPU.

@cwsmith cwsmith marked this pull request as ready for review November 6, 2025 18:53
@cwsmith cwsmith requested a review from bartgol November 6, 2025 19:26
Copy link
Collaborator

@bartgol bartgol left a comment

Choose a reason for hiding this comment

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

Looks good!


#include <Panzer_IntrepidFieldPattern.hpp>

//FIXME!
Copy link
Collaborator

Choose a reason for hiding this comment

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

What's the FIXME about? Meaning, what is wrong with the code below that we need to keep in mind?

Copy link
Collaborator Author

@cwsmith cwsmith Nov 7, 2025

Choose a reason for hiding this comment

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

Thanks for asking; I forgot about this. Is there a better/cleaner way to use the exec/mem space that Albany is using?

Copy link
Collaborator

Choose a reason for hiding this comment

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

In Albany_KokkosTypes.hpp there are some types that may help. It all comes out of PHX::Device.

set_target_properties(landIce PROPERTIES PUBLIC_HEADER "${HEADERS}")
target_link_libraries(landIce PUBLIC albanyLib)
if (ALBANY_OMEGAH)
target_link_libraries(landIce PUBLIC Omega_h::omega_h)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am not sure why we link omegah to landIce. We don't explicitly use omegah stuff in landice, and it is already linked indirectly through albany lib.

I think we can remove these two lib linking...

# 2d test
configure_file(${CMAKE_CURRENT_SOURCE_DIR}/input_2d_omegah.yaml
${CMAKE_CURRENT_BINARY_DIR}/input_2d_omegah.yaml COPYONLY)
add_test(${testName}_2d_omegah ${SerialAlbany.exe} input_2d_stk.yaml)
Copy link
Collaborator

Choose a reason for hiding this comment

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

whoops! Good catch!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants