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

Updates to use latest Trilinos develop branch #1255

Merged
merged 7 commits into from
Apr 16, 2024

Conversation

jrood-nrel
Copy link
Contributor

No description provided.

@jrood-nrel jrood-nrel requested review from psakievich and alanw0 April 12, 2024 20:00
Copy link
Contributor

@alanw0 alanw0 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, a bunch of search/replace and add prefix...
The other API changes you encountered went through a deprecation cycle where they would have been a warning for at least 6 weeks before they were actually changed in a way that causes an error. Sometimes we debate whether we should lengthen the deprecation window to be more gentle on users that don't update as often...

src/Realm.C Show resolved Hide resolved
src/Realm.C Show resolved Hide resolved
@psakievich
Copy link
Contributor

Just a reminder that this will break all of our default builds so we will need to add versions and update dependencies in spack right away.

@jrood-nrel
Copy link
Contributor Author

Yeah I will get other stuff in place before merging this. I'm mostly curious if it will help me with the segfaults on Aurora.

@jrood-nrel jrood-nrel marked this pull request as ready for review April 15, 2024 18:17
@jrood-nrel jrood-nrel merged commit da0d9f6 into Exawind:master Apr 16, 2024
1 of 3 checks passed
@jrood-nrel jrood-nrel deleted the jrood/update-stk-throw branch April 16, 2024 19:55
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.

3 participants