Skip to content

Conversation

@grnydawn
Copy link

@grnydawn grnydawn commented Oct 2, 2024

A bug fix to resolve the issue #143 related to logging for Kokkos data types.

Tested and passed on Chrysalis, pm-cpu, pm-gpu, and Frontier.

  • Testing
    • CTest unit tests for new features have been added per the approved design.

Copy link

@philipwjones philipwjones left a comment

Choose a reason for hiding this comment

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

Looks fine and passes testing.

Copy link
Member

@mwarusz mwarusz left a comment

Choose a reason for hiding this comment

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

All tests compiled and passed on pm and frontier for both cpu and gpu configurations. LGTM. Thanks @grnydawn.

@brian-oneill
Copy link

Yep, that fixed it for me too. Built and passed tests on pm-cpu and pm-gpu

@grnydawn grnydawn force-pushed the ykim/omega/logging branch from bd287c4 to 31e5036 Compare October 3, 2024 15:56
@philipwjones philipwjones merged commit 1cf022b into E3SM-Project:develop Oct 3, 2024
2 checks passed
@mark-petersen mark-petersen removed their request for review October 4, 2024 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants