[Kokkos] trailing whitespace, indentation fixes
[Kokkos] assortment of small fixes

Mostly uncaught exceptions, but also uninitialized variables,
and division-by-zero checks.

Build/Test Cases Summary
Enabled Packages: KokkosClassic
Enabled all Forward Packages
0) MPI_DEBUG => passed: passed=771,notpassed=0 (32.49 min)
1) SERIAL_RELEASE => passed: passed=695,notpassed=0 (12.90 min)
Other local commits for this build/test group: 260b8b2
Added destructors for properly deallocating memory

Build/Test Cases Summary
Enabled Packages: Epetra, FEI, Isorropia
Enabled all Forward Packages
0) MPI_DEBUG => passed: passed=975,notpassed=0 (14.18 min)
1) SERIAL_RELEASE => passed: passed=902,notpassed=0 (6.21 min)
MueLu: Ifpack2Smoother: added missing fwd declaration for BlockedCrsMatrix
MueLu: Q2Q1: only need to reset debug data when MueLu_ENABLE_DEBUG is on
MueLu: Q2Q1: fixing warnings
Attempt to fix possible bug when repartitioning is used and a processor calls List.set('x-coordinates',NULL);
Moving the construction of the auxiliary nodal matrix; this needs to be recomputed
every time the system matrix changes.
Tpetra: Fixed test build error with new Kokkos and CUDA.
Tpetra: Fixed build warnings (unused typedef, signed vs. unsigned comp)
Kokkos (Classic): Fixed complex-arithmetic build error.

Whoever added an assert with fabs() in the sparse triangular solve
routines: Remember that they also have to build for Scalar =
std::complex<T> :-) If you have ETI enabled, you may enable the
following CMake options in order to run some tests for complex Scalar:

  Teuchos_ENABLE_COMPLEX:BOOL=ON
  Tpetra_INST_COMPLEX_DOUBLE:BOOL=ON

Thanks!
KokkosClassic: Fixed build warnings and formatting.

Build/Test Cases Summary
Enabled Packages: KokkosCore, KokkosCompat, KokkosLinAlg, KokkosContainers, KokkosMpiComm, Tpetra
0) MPI_DEBUG => passed: passed=73,notpassed=0 (1.61 min)
1) SERIAL_RELEASE => passed: passed=54,notpassed=0 (0.07 min)
2) MPI_RELEASE_KOKKOS_REFACTOR => passed: passed=76,notpassed=0 (14.25 min)
Other local commits for this build/test group: 6b88a2c, 877f4b1, 81b686a
MueLu: Utils: adding python ssh util
ML: fix MPI test hang on two processors

Commit 258fe253 caused the ML_Communicator test to hang.  I think this
was actually a bug waiting to happen:

  An ML preconditioner was created only on processor 0, but using communicator MPI_COMM_WORLD.
  ML_gsum_int was called using MPI_COMM_WORLD.
  hang ...

It seems amazing we haven't hit this previously. It's possible that the communicator was
replaced at some later point in the computation, but I didn't go looking for this.  Anyway,
my fix is just to replace MPI_COMM_WORLD immediately with the communicator of the incoming fine grid matrix.

Build/Test Cases Summary
Enabled Packages: ML, MueLu
Enabled all Forward Packages
0) MPI_DEBUG => passed: passed=257,notpassed=0 (25.63 min)
1) SERIAL_RELEASE => passed: passed=229,notpassed=0 (13.95 min)
Other local commits for this build/test group: e4d9200
MueLu: Ifpack2Smoother: minor cleanup

- Removed extra unnecessary includes
- Changed var names to more standard names in MueLu
Kokkos:  Fix CRS mat-vec for rank-2 views with dimension_1() == 1.

After doing some profiling between TrilinosCouplings/FENL and
Kokkos/FENL, I discovered mat-vec's with rank-2 views but with
dimension_1() == 1 were calling the rank-2 kernel.  Thanks to Christian
for pointing out what needed to be fixed.
Kokkos:  Capture mat-vec times in FENL and add vtune command line arg.

This captures mat-vec times in the CG iteration and displays them.  I
also added code to connect FENL to vtune by adding the "vtune" command
line argument.  Currently the path to vtune is hard-coded.
TrilinosCouplings:  Add code for connecting VTune to FENL.

Just add --vtune to your command line arguments to profile with vtune.
Currently the path to vtune is hard coded.
MueLu: renamed UncoupledAggregationAlgorithm -> AggregationPhase1Algorithm
MueLu: UC: algorithm base cleanup

Main change: removed NodeState namespace