Trilinos Master Merge PR Generator: Auto PR created to promote from master_merge_20240830_175836 branch to master #13418
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
SpAdd handle: delete sort_option getter/setter (#2296) SpAdd handle was originally a copy-paste of the spgemm handle way back in #122, and included get_sort_option() and set_sort_option() from spgemm. But these try to use the member bool sort_option, which doesn't exist. Somehow these functions never produced compile errors until someone tried to call them. Fixes build errors on Sunspot.
…p-serialize-io-bug Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'IOSS: Fix serialize io issue in timestep query' PR Author: gsjaardema
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.26.3 to 3.26.5. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@883d858...2c779ab) --- updated-dependencies: - dependency-name: github/codeql-action dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Phalanx: fix hip issue, add test for UniqueToken' PR Author: rppawlo
Kokkos compatibility update following kokkos/kokkos#7256
Sacado: Band-aid fix for LayoutContiguous
…tions/develop/github/codeql-action-3.26.5 Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Bump github/codeql-action from 3.26.3 to 3.26.5' PR Author: dependabot[bot]
sacado: update Kokkos_ViewMapping.hpp location
…bly (#13391) Intrepid2: This PR adds support for additional uses cases to the existing sum-factorized assembly mechanism. On the way, it adds support for vector weights to `TransformedBasisValues`; previously, only matrix and scalar weights were supported. This PR also makes some minor adjustments to the timings performed in `StructuredIntegrationPerformance.cpp` to ensure fair comparisons. The new use cases are demonstrated most directly in the following: ``` packages/intrepid2/assembly-examples/VectorWeightedGRADGRADStructuredAssembly.hpp ``` which demonstrates performing an integral of the form $`({\mathbf a} \cdot \nabla e_i, {\mathbf b} \cdot \nabla e_j)`$ for an $H^1$ basis. ## Testing The new use cases are well-exercised by the tests; a new set of `VectorWeightedPoisson` test cases (corresponding to the integral above) are included in the structured-versus-standard tests. ## Other notes/details: - Added support for dot products to Intrepid2_Data; added a corresponding test. Also hardened the test MatVec_CPDD_transpose to include a check that u' A v = v' A' u for vectors u, v. - Added free function rank() taking BasisValues object as argument. - Data: fixed an issue in allocateMatVec in which an incorrect variation type and/or incorrect extent could be used for the final result dimension. - TransformedBasisValues: added support for a (C,P,D) transform, with the main use case being a dot product with a vector-valued basis evaluation. - In allocateMatMatResult(), fixed an issue in which the wrong getUnderlyingView() method was being called; resolved by calling the one that gets a DynRankView. - Added a test against taking the outer product of two vectors; fixed the issue that this demonstrated in Intrepid2::Data. - Added test template StructuredVersusStandardVectorWeighted_D2_P1_P1. - Fixed an issue with the transpose arguments to a mat-mat call. - In setJacobianDetInv(), corrected argument name in method declaration and doxygen. - In DataTools, broadened the use cases for multiplyByCPWeights(), and added a transposeMatrix() method. - In TransformedBasisValues, fixed some issues with spaceDim(). - In StandardAssembly and StructuredAssembly, revised to support more vector-weighted use cases. - Added more vector-weighted tests, covering cases when a vector field is dotted with a vector and then integrated against a scalar. - Modified standard assembly performance tests to exclude the orientation application from "core integration" timing. - Added VectorWeightedPoisson to allFormulationChoices. For now, setting the "best" CUDA choices to match Poisson. But we need to redo those calibrations regardless, and maybe switch to reading them in from file: these are for older CUDA cards.
The changes exactly match github.com/kokkos/kokkos-kernels/pull/2293. Improves performance of sort_crs_matrix by up to 6.3x.
c++20 added a new keyword "requires" for concepts. Phalanx had a function on a templated class called requires...
Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'RTOps,Phalanx: fixes for cpp20' PR Author: rppawlo
…dded-fmt-version Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Seacas update embedded fmt version' PR Author: gsjaardema
Signed-off-by: Greg Sjaardema <[email protected]>
Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Changes for building with c++20' PR Author: gsjaardema
Modify the expected instance counts for OpenMP in CrsMatrix_MatvecFence.cpp Compatibility update for KOKKOS_VERSION >= 40499, corresponding to kokkos/kokkos#7245
tpetra: modify expected instance fence counts in MatvecFence
For intel icpc compilers use inline constexpr rather than static constexpr for is_same_v specialization introduced in #13392 This is to workaround intel ICE issues that should not be related to the change but occur when building Trilinos with or without Kokkos' develop branch Error output sample: ``` /home/ndellin/trilinos/Trilinos/packages/sacado/test/UnitTests/ConditionalReturnTypeTest.cpp(48): internal error: assertion failed at: "il.c", line 10434 in trans_unit_for_source_corresp ```
Change MueLu to use <packageName>_SHOW_DEPRECATED_WARNINGS instead of TRILINOS_HIDE_DEPRECATED_HEADER_WARNINGS. Now we can simplify code in the configuration management tool. Signed-off-by: Samuel E. Browne <[email protected]>
While we're dealing with the Epetra stack deprecation, it's too high of a noise:signal ratio to have the deprecation warnings enabled in CI testing. Signed-off-by: Samuel E. Browne <[email protected]>
KokkosKernels: patch #2296
…eader-warnings-for-pr Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'Framework: Disable deprecated header warnings for pr' PR Author: sebrowne
…-icpc-workaround Automatically Merged using Trilinos Pull Request AutoTester PR Title: b'sacado: workaround icpc ice with is_same_v' PR Author: ndellingwood
KokkosKernels: improve CRS sorting performance
|
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: PR_gcc-openmpi-openmp
Jenkins Parameters
Build InformationTest Name: PR_gcc
Jenkins Parameters
Build InformationTest Name: PR_gcc-openmpi_debug
Jenkins Parameters
Build InformationTest Name: PR_clang
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_python3
Jenkins Parameters
Build InformationTest Name: PR_cuda
Jenkins Parameters
Build InformationTest Name: PR_intel
Jenkins Parameters
Build InformationTest Name: PR_cuda-uvm
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: PR_gcc-openmpi-openmp
Jenkins Parameters
Build InformationTest Name: PR_gcc
Jenkins Parameters
Build InformationTest Name: PR_gcc-openmpi_debug
Jenkins Parameters
Build InformationTest Name: PR_clang
Jenkins Parameters
Build InformationTest Name: Trilinos_PR_python3
Jenkins Parameters
Build InformationTest Name: PR_cuda
Jenkins Parameters
Build InformationTest Name: PR_intel
Jenkins Parameters
Build InformationTest Name: PR_cuda-uvm
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# 13418: IS A SUCCESS - Pull Request successfully merged |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Auto PR created to promote from master_merge_20240830_175836 branch to master