1. do call postsmooth instead of pre smoothing for post smoothing.
2. Prolongate coarse grid correction to correct fine level vector.
3. Changed test to use regular patches and schwarz mod that mimics
   regular smoother for better comparison.
4. Improved documentation.

Previously setting the define overwrote the flags for superlu.

With this addition it is possible to construct a twolevel method by
specifying a fine level smoother (e.g. SeqOverlappingSchwarz), a
policy for the grid transfer, and a policy for constructing the
coarse level solver.

This patch removed the tests for umfpack from dune-istl as they
are already needed in dune-common. There are modules using umfpack
that do not require dune-istl. Therefore dune-common seems to be the
more appropriate place for the test.

Arne Morten Kvarving authored 11 years ago and Markus Blatt committed 11 years ago

now the preconditioner can be used in a Dune::Preconditioner context

Dominic Kempf authored 11 years ago and Markus Blatt committed 11 years ago

This reverts commit 5336f23b950e9f1b5b8a51cdc103142ab47a779e.

This is supposed to be a temporary hack. If Umpack is available
the it is chosen of SuperLU. Otherwise SuperLU is chosen if it
is available.

Dominic Kempf authored 11 years ago and Markus Blatt committed 11 years ago

This commit contains
- a wrapper for the C library UMFPack
- tests for both CMake and autotools to find UMFPack on the system
- a unit test for UMFPack
- a new MatrixType ColCompMatrix which is a base class for a column
  compressed matrix. SuperLUMatrix now inherits from this base class.
  This way, no code is duplicated for the very similar interface in
  UMFPack.
- the SuperLU-specific part of OverlappingSchwarz code is abstracted
  to work with either SuperLU or UMFPack

This caused build errors with CMake.

fix "unused variable" warnings by proper use of #ifdef

MatrixAdapter uses a const reference to store a reference
to the matrix it is working on. Without this patch it was
possible to pass the matrix to constructor actually taking
a MatrixAdapter as its argument. This resulted in a const
reference to a temporary MatrixAdapter that died after the
constructor call.

This patch request explicitly calling the constructor and
thus fixes this issue.

Elias Pipping authored 11 years ago and Markus Blatt committed 11 years ago

This problem was introduced in 3da29f78

The last patch missed one instance of double.
# Please enter the commit message for your changes. Lines starting
# with '#' will be ignored, and an empty message aborts the commit.
# On branch master
# Changes to be committed:
#   (use "git reset HEAD <file>..." to unstage)
#
#	modified:   ../../../../dune/istl/test/superlutest.cc
#
# Untracked files:
#   (use "git add <file>..." to include in what will be committed)
#
#	../../../../build-clang/
#	../../../../build-test/
#	../../../../parallel/
#	../../../../test-shared/

Somehow I missed this instance in the transition.

Previously this code was plain wrong. It actually made a copy
of the vector and tried to swap its entries with the original vector.
The argument to swap even was a const reference!
Now we use the correct trick: vector().swap(other).

use FieldTraits to determin the return type of the norm methods

use FieldTraits to determin the return type of the norm methods