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

chore: upgrade Trilinos version #288

Merged
merged 13 commits into from
Dec 11, 2024
Merged

chore: upgrade Trilinos version #288

merged 13 commits into from
Dec 11, 2024

Conversation

CusiniM
Copy link
Contributor

@CusiniM CusiniM commented Nov 21, 2024

The previous version was giving me some compilation errors with clang16 on macOS so I just upgraded it.

@CusiniM CusiniM self-assigned this Nov 21, 2024
@CusiniM
Copy link
Contributor Author

CusiniM commented Nov 22, 2024

well, Epetra, which we use, is deptrecated so we need to clean up our interface and switch to Tpetra. They do have cmake options to still use it but I feel that we should just clean this.

@OmarDuran
Copy link

well, Epetra, which we use, is deptrecated so we need to clean up our interface and switch to Tpetra. They do have cmake options to still use it but I feel that we should just clean this.

Thank you @CusiniM for fixing this. I have tested with apple clang 16.0 and the compilation errors are fixed. However, notice that the action Rockylinux (8, clang 17.0.6, cuda 12.5) is failing.

@rrsettgast rrsettgast merged commit db12f81 into master Dec 11, 2024
10 checks passed
liyangrock added a commit to liyangrock/thirdPartyLibs that referenced this pull request Dec 12, 2024
* chore: upgrade Trilinos version

* enable zoltan (GEOS-DEV#287)

* enable zoltan

* specify clang17 in rockylinux build

* exclude clang from update after install

* update caliper to avoid error with std::filesystem linker errors

* try updating cuda image to 12.6.3

Co-authored-by: Matteo Cusini <49037133+CusiniM@users.noreply.github.com>
Co-authored-by: Randolph Settgast <settgast1@llnl.gov>
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