Skip to content

MueLu: clean-up MueLu_AlgebraicPermutationStrategy_def.hpp for #3492

James Willenbring requested to merge lucbv:MueLu_fix_#3492 into develop

Created by: lucbv

@trilinos/muelu

Description

Changed the type of some variables from Scalar to Teuchos::ScalarTraits<Scalar>::magnitudeType to avoid issues related reductions using: REDUCE_MIN/MAX operations that are not well defined when Scalar=std::complex<>.

Motivation and Context

This cleans-up the code and potentially fixes a bug

Related Issues

How Has This Been Tested?

A local build was done with Scalar=std::complex<> has been performed

Checklist

  • My commit messages mention the appropriate GitHub issue numbers.
  • My code follows the code style of the affected package(s).
  • I have read the code contribution guidelines for this project.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • No new compiler warnings were introduced.

Merge request reports