-
Notifications
You must be signed in to change notification settings - Fork 579
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
Trilinos Master Merge PR Generator: Auto PR created to promote from master_merge_20240202_175839 branch to master #12717
Conversation
Tempus has had a required test dependency on Epetra, Belos and Stratimikos that actually was an optional dependency in which only a couple test needed them. This cleans this up so that most tests will still be run when Epetra, Belos and Stratimikos (and Tpetra) are not available. Notes: * Created combined options TEMPUS_ENABLE_EPETRA_STACK and TEMPUS_ENABLE_TPETRA_STACK that are true when Epetra or Tpetra are available along with -D Tempus_ENABLE_(E|T)petra:BOOL=ON -D Tempus_ENABLE_Thyra(E|T)petraAdapters:BOOL=ON -D Tempus_ENABLE_Belos:BOOL=ON -D Tempus_ENABLE_Stratimikos:BOOL=ON -D Stratimikos_ENABLE_Belos:BOOL=ON * Only CDR tests in BackwardEuler and BDF2 required Epetra or Tpetra so limited dependencies to those two tests. * Removed spurious includes for Epetra, Tpetra, Stratimikos and others.
Belos has native Tpetra linear solvers that should not be registered with the Stratimikos solver factory when Tpetra is not enabled. This has been fixed. During some testing there were clear errors in the cmake logic that enables or disables Stratimikos tests based on what packages are enabled. These resulted in configure-time errors from cmake or build errors. While not exhaustive, several logical errors were fixed in this commit.
…ameter list Belos has native Tpetra linear solvers that should not be registered with the Stratimikos solver factory when Tpetra is not enabled. This includes the solver names in the parameter list, which were missed in the previous commit. This has been fixed. The Epetra-only Stratimikos tests are now passing.
There is an implicit dependency on a preconditioner (Ifpack) for some of the Belos tests. Without Ifpack, several of the tests will fail. A "FIXME" was left in the CMakeLists.txt file noting this issue, which has now been addressed.
PR kokkos/kokkos-kernels#1962 introduced API updates to the spadd_* routines This is allowed as spadd resides in the Experimental namespace, but requires propagating backward-compatibility breaking changes to Trilinos Resolve kokkos/kokkos-kernels#2089
Add missing guards for KOKKOSKERNELS_VERSION
Origin repo remote tracking branch: 'github/master' Origin repo remote repo URL: 'github = [email protected]:TriBITSPub/TriBITS.git' Git describe: tribits_start-3375-g9328b340 At commit: commit 2f17d8e766e950378a3d0690ec28973134be7d62 Author: Roscoe A. Bartlett <[email protected]> Date: Wed Jan 24 19:38:16 2024 -0500 Summary: Merge branch 'master' into 597-config-git-version-info
…arent-info (TriBITSPub/TriBITS#597) Main purpose is to pull in the TriBITS PR for getting the git repo parent commit info: * TriBITSPub/TriBITS#598
…riBITSPub/TriBITS#597) This is needed to know the versions of the target and topic branches for PR builds submitted to CDash. This is needed to be able to detect random test failures.
MueLu LWGraph_kokkos unit test: Fix bug
…t-version-parent-info Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Update TriBITS snapshot 2024-01-24 and set Trilinos_SHOW_GIT_COMMIT_PARENTS=ON for PR builds' PR Author: bartlettroscoe
Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Teuchos/Panzer: add FOM in mini-em for acceptance testing' PR Author: rppawlo
…orter Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'STK: Implement Reporter without virtual' PR Author: masterleinad
tpetra: update spadd_* api for KOKKOSKERNELS_VERSION >= 40299
A few asserts were using the wrong views. There were many parallel_fors that were using the default exec space even when ExecSpace for the class was set to Serial. This was causing GPU kernels to try to access Serial views
…_issues Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Fix Shylu GPU/Serial issues' PR Author: jgfouca
Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Framework: Remove complex from all declared non-complex configs' PR Author: sebrowne
Insure that nobody commits their own .pre-commit-config.yaml by accident
… have the clang module loaded
Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'MueLu: Adding alternative clang-format script' PR Author: csiefer2
Tempus and Stratimikos: Cleanup Epetra/Tpetra Dependencies
- Load correct CMake version for both arm-20 and arm-22 - For arm-20, most TPLs were removed from sparc-dev so load them manually. Trilinos perf test build (which enables all the common TPLs) configs and builds cleanly with both arm-20 and arm-22 now.
Fix Stria (van1-tx2) ATDM environments
The machine testing.sandia.gov was moved and they changed the firewall to not accept http (port 80) submits. Those were just ignored (like sending ouptut to /dev/null). The moved machine testing.sandia.gov must use https (port 443). Also, I had to unset the proxy vars so that CTest does not try to use a proxy by running: unset HTTP_PROXY unset http_proxy unset https_proxy unset HTTPS_PROXY unset all_proxy unset ALL_PROXY And with this, I was also able to submit to testing-dev.sandia.gov/cdash as well using https. (The machine testing-dev.sandia.gov was moved a long time ago and apparently had its http submit turned off.)
…02-01 Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Change from http to https to submit to moved testing.sandia.gov/cdash' PR Author: bartlettroscoe
Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request. |
Status Flag 'Master Merge AutoTester' - Testing Jenkins Projects: Pull Request Auto Testing STARTING (click to expand)Build InformationTest Name: Trilinos_PR_gcc-8.3.0
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_gcc-8.3.0-serial
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_gcc-8.3.0-debug
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_clang-11.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_cuda-11.4.2-uvm-off
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_intel-2021.3
Jenkins Parameters
Using Repos:
Pull Request Author: trilinos-autotester |
Status Flag 'Master Merge AutoTester' - Jenkins Testing: all Jobs PASSED Pull Request Auto Testing has PASSED (click to expand)Build InformationTest Name: Trilinos_PR_gcc-8.3.0
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_gcc-8.3.0-serial
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_gcc-8.3.0-debug
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_clang-11.0.1
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_cuda-11.4.2-uvm-off
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_intel-2021.3
Jenkins Parameters
|
Status Flag 'Pre-Merge Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request. |
Status Flag 'Master Merge AutoTester' - Pull Request will be Automerged |
Merge on Pull Request# 12717: IS A SUCCESS - Pull Request successfully merged |
Auto PR created to promote from master_merge_20240202_175839 branch to master