Skip to content

Conversation

@e10harvey
Copy link
Owner

No description provided.

e10harvey and others added 30 commits May 10, 2023 12:30
docs: Remove unsupported Doxyfile.in tags
The new test
`TrilinosInstallTests_simpleBuildAgainstTrilinos_by_package_build_tree` merged
from PR #11863 fails because the subdirs ${CMAKE_CURRENT_BINARY_DIR}/common
and ${CMAKE_CURRENT_SOURCE_DIR}/common because this CMakeLists.txt file
already sits in the kokkos-kernels/common/ subdir.

I don't know why this error did not happen with PR testing for PR #11863 but
this is clearly the right thing to do.
Required for for TriBITS-compliance
Discussion at trilinos/Trilinos#8866 (comment)
lucbv and others added 28 commits July 30, 2023 11:58
remove triplicate sanity checks in BsrMatrix
The solver was implemented along with some helper
utilities: solver status and solver params.
Four tests are added to check the interface, the
correctness of single solves and the correctness
in parallel solves that mimic its use in FE/FD code.
remove Intel 2017 code (no longer supported)
(HashmapAccumulator data races on Ada and Hopper architectures).
To avoid checking for every architecture at every location, add a
new macro KOKKOSKERNELS_CUDA_INDEPENDENT_THREADS which is defined
if we're targeting an architecture with independent thread scheduling.
remove duplicate BSR SpMV tests
SpAdd perf test can call cusparse and MKL TPLs directly, but this can
only be done if the default ordinal and offset type are both int.
Check for this ahead of time and print a decent error message.

Fixes build error when cusparse enabled, but offset int not enabled.
(insert a space between it and previous flag)
Fix SpAdd perf test when offset/ordinal is not int
Attempt to resolve "error: ‘pair’ was not declared in this scope" in
cuda/11.2+gcc/8.3.0 builds
Test_ODE_Newton: Add template parameters for Kokkos::pair
Also on ger():
- better explanations on unit_tests,
- its unit tests now have terminology similar to syr() unit tests,
- replace 'one level' and 'two level' terminology on the impl file, following syr().
Also on syr():
- small improvement on the ROC tpl file
Remove printouts from the unit tests of ger() and syr()
including cusparse, rocsparse, onemkl wrappers.

Fix some misc. bugs too:
- Fallback check for cusparse was wrong - we were only calling the TPL
for mode 'N', but TPL should be used for 'T' and 'H' also
- In SYCL build, forgetting to #undef a macro caused warning in sparse
tests
- Can't call cuSPARSE or MKL spmv with mode 'H' and real scalars. In
this case, just switch to mode 'T' since it's equivalent
(no longer needs to match AMatrix::execution_space)
Move them from KokkosSparse:: to KokkosSparse::Impl::
since they're not supposed to be part of the public interface.
Keep an alias in KokkosSparse:: for backwards compatibility, but mark it
[[deprecated]].
e10harvey pushed a commit that referenced this pull request Sep 6, 2023
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.