The standard states (8.3.6/4) that adding default arguments in later
function declarations are only valid for non-template functions.
In matrixmarket.hh, the function template loadMatrixMarket is declared
and a default argument for the last parameter is provided. In
owneroverlapcopy.hh, OwnerOverlapCopyCommunication contains a friend
declaration of loadMatrixMarket without the default parameter. If now
owneroverlapcopy.hh is included before matrixmarket.hh, the friend
declaration is the first declaration and the default parameter in
matrixmarket.hh is invalid.
By including matrixmarket.hh in owneroverlapcopy.hh, we make sure that
the declaration which provides the default argument is always the first
one.

This test actually does test the parallel functionality
that is now contained in dune-common for quite some time.
While it is using BlockVector this is not needed by the test.
It works with any container supporting operator[].

Therefore this commit gets rid of BlockVector and uses std::vector
instead. This will allow us to move the test to dune-core subsequently.

Previously, we assumed that if a_{ij} is stored in the sparse matrix
then a_{ji} must be stored also and used mat[i][j] to access. If the entry
was not stored then an exception somewhere in basearray.hh was thrown.
Unfortunately knowing the cause for this exception is insider knowledge,
seldomly leaked ;). In addition there is now reason to throw an expection here.

With this commit we the find method on the row to search for the entry a_{ji}.
If it is not present we simply treat it as being zero.

Fixes FS#FS#1649.

extended the amgtest to also use field_type = complex by switching the
definition of XREAL

instead of just returning the real part, we return csgn(v)*abs(v). For real values this is
equivalent to returning v.

a) add the appropriate typedefs for real_type
b) update the norm implementations to return real_type
c) change all variables assigned from norm_(...) to real_type

we have to retrieve an int which is stored in field_type. If the field_type is std::complex, we
have to access the real-component, if it is double/float, we don't have to do anything
special. We use std::real(.) to get the correct real value. This function (with its appropriate
overloads) is available from c++11 on, but this part was already supported by the ancient
g++ version we supprt ;-)

on fedora 21, the default subdirectory for SuperLU includes is
called "SuperLU".

Emmanouil Kiagias authored 9 years ago and Christoph Grüninger committed 9 years ago

Updated how METIS_PartGraphRecursive is called in METIS version 5.
In the new parameters NULL value is used where possible. Only one
new parameter could not be set to NULL, it is the "ncon" (second argument).
In the manual it says: "ncon: The number of balancing constraints.
It should be at least 1." so 1 is used as its value.

Also options with the new layout are passed in METIS_PartGraphRecursive
initialized with the default values(default values also used in the
current code: options[0]=0). Only one option was added manually:
the METIS_OPTION_NUMBERING = numflag which is the same numflag parameter
passed in the old version of METIS_PartGraphRecursive function.

Signed-off-by: Christoph Grüninger <gruenich@dune-project.org>

On request of Markus Blatt.  Supposedly there have been problems with and/or on
older compilers.

When DUNE_ISTL_WITH_CHECKING is enabled, the BCRSMatrix throws an
exception when trying to access a row that has no columns. This creates
a problem in PDELab when using nested BCRSMatrices (as some people do
for operator splitting in PDELab), where an off-diagonal BCRSMatrix
block might be completely empty (i.e. have a correctly set up structure,
but every row in the off-diagonal BCRSMatrix is empty. If the PDELab
assembler then attempts to set a Dirichlet constraint, it has to access
the row to set all columns in that row to zero, but the check in
operator[] prevents it from doing so.

So this patch just drops that check.

This patch removes the assumption that the field_type and / or real_type
can always be implicitly converted to double and can be implicitly
constructed from an int (in constructs like field_type d = 0;).

This patch is required for compatibility with the FLOP counter in PDELab.

It belongs to dune-common's config.h.cmake. Otherwise
it cannot be shared with other modules using ParMETIS.

It should be std::endl.
Courtesy of GCC 5-svn, it refuses to compile these.

While c++11 allows a trailing comma in enums,
we should avoid this because it provokes
gcc-4.4 warnings.

The check gets removed from dune-common, thus ISTL has to call it itself.

This solves warnings in UG for redefined macros.
Thanks to Christian for hinting the proper way
fixing this.

Jonathan Youett authored 10 years ago and Markus Blatt committed 10 years ago

When assigning a BCRSMatrix to another BCRSMatrix which was setup for
a different size/pattern, it segfaulted and I got the following
Exception:

"InvalidStateException [allocate:.../dune/istl/bcrsmatrix.hh:2108]:
Rows have already been allocated, cannot allocate a second time."

The problem was, that inside the assignment operator the rows (r) were
never deallocated. This commit fixes this and adds a testcase.

Testcase slightly modified by Markus.

This mode is not implemented and therefore throws
an assertion that makes the test fail everytime.