Merge pull request #3468 from drwells/pvs-integration-info-initialize

Initialize an IntegrationInfo field.

Initialize an IntegrationInfo field.

This number is correctly filled by IntegrationInfo::initialize, but this
way it starts out with a bogus value (instead of an undefined one).

Merge pull request #3462 from bangerth/rename-variables-2

Rename some variables.

Merge pull request #3463 from bangerth/rename-variables-3

Rename some variables.

Merge pull request #3464 from bangerth/rename-functions-and-variables

Rename local variables with names equal to members of the base class.

Merge pull request #3465 from bangerth/clarify-code

Clarify the intent of a piece of code.

Merge pull request #3466 from bangerth/initialize-variable-5

Initialize a member variable of LogStream.

Merge pull request #3467 from bangerth/initialize-variable-6

Initialize member variables.

Merge pull request #3459 from bangerth/rename-arguments

Rename function arguments.

Merge pull request #3458 from bangerth/rename-variables

Rename some variables.

Initialize member variables.

Initialize a member variable of LogStream.

The variable really was never initialized, even though some
output depended on it. I suspect that it just happened to always
be initialized to 'false'. Make this explicit.

Clarify the intent of a piece of code.

The code uses integer division, followed by a double precision multiplication.
It is admittedly not immediately obvious what it is supposed to do. Clarify
that we really mean the integer division.

Rename local variables with names equal to members of the base class.

This is certainly confusing.

While there, notice that the functions that *compute* quadrature points
and weights are called 'set_*', but are 'const'. That makes no sense.
In fact, they just compute and return these values, so call them
'get_*' instead.

Rename some variables.

There are variables of the same name in the base class. Avoid the
confusion of having similarly named variables in the local scope.

Rename some variables.

There are variables of the same name in the base class. Avoid the
confusion of having similarly named variables in the local scope.

Merge pull request #3450 from bangerth/fix-warning

No need for double parentheses.

Merge pull request #3446 from bangerth/fix-DataOutRotation-build_one_patch

Rename a function argument.

Merge pull request #3453 from bangerth/initialize-variables-4

Initialize member variables.

Merge pull request #3338 from kalj/check-for-compute-capability

Check for compute capability

Merge pull request #3461 from kalj/patch-1

Use a single if block

Use a single if block

Fix warning from PVS about two if blocks with identical condition. Fixes #3369.

Rename function arguments.

The Nedelec class used 'degree' in a few places, but it was
really the 'order' of the element. The polynomial degree as
used in the base class is 'order+1'.

Rename some variables.

There are variables of the same name in the base class. Avoid the
confusion of having similarly named variables in the local scope.

Merge pull request #3455 from masterleinad/fix_parpack_test

Fix parpack test

Merge pull request #3448 from davydden/matrix_free_Laplace_GMG_unit_test

add another matrix-free laplace GMG unit test

Merge pull request #3449 from drwells/pvs-timer-initialization

Add some missing initializations to Timer.

Fix parpack test

Initialize member variables.

Merge pull request #3452 from bangerth/initialize-variables

Initialize member variables.

Initialize member variables.

Rename a function argument.

DataOutRotation::build_one_patch took a variable named 'patches',
but unlike in #3445, it is not a reference to the class's member
variable of the same name. Rather, it is an array this function
needs to fill in, and nobody else sees it. Consequently,
rename it 'my_patches'.

While there, also update the documentation, which is no longer
in agreement with the implementation.

No need for double parentheses.

do not require Trilinos for matrix_free tests which do not use it

add another matrix-free laplace GMG unit test

Add some missing initializations to Timer.

Some of these were caught by PVS studio.

Merge pull request #3445 from bangerth/dont-duplicate-variable

Let DataOut::build_one_patch use a member variable.

Merge pull request #3444 from class4kayaker/static_fix

Fix FE_QHier static analysis warning

Merge pull request #3434 from bangerth/initialize-member-variable

Initialize a member variable.

Merge pull request #3435 from bangerth/initialize-member-variable-2

Initialize member variables to invalid values.

Let DataOut::build_one_patch use a member variable.

Currently, it takes a reference to a variable as argument, but this variable
is a member variable of the class to which it has access. That's duplicative.
To make things even more confusing, the local variable and the member
are named the same.

Fix this by just not taking the reference as argument, and directly
accessing the member of the class.

Merge pull request #3439 from bangerth/simplify-computation

Simplify an initialization.

Merge pull request #3437 from bangerth/be-careful-with-streams

Be more careful with streams.

Merge pull request #3436 from bangerth/avoid-pointless-condition

Avoid a pointless comparison.

Fix FE_QHier static analysis warning

Fix to #3406

Merge pull request #3442 from Rombur/add-assertthrow

Add a forgotten assert.

Add a forgotten assert.

Merge pull request #3441 from bangerth/initialize-variables-2

Initialize a member variable to invalid values.

Merge pull request #3431 from bangerth/make-variable-names-uniform

Make variable names uniform.

Merge pull request #3432 from bangerth/fix-blockvectorbase-memory_consumption

Fix BlockVectorBase::memory_consumption.

Initialize a member variable to invalid values.

Simplify an initialization.

Be more careful with streams.

In particular, stop also if in.fail() is true, not just if in.eof() is true.
I suspect that it matters little in practice, but who knows.

Avoid a pointless comparison.

The function works in 3d, so checking for 'dim>1' makes no difference.

Initialize member variables to invalid values.

Initialize a member variable.

Merge pull request #3433 from bangerth/fix-DataOut-memory_consumption

Update memory consumption output.

Merge pull request #3394 from drwells/pvs-function-lib-warnings

Silence a PVS studio warning about repeated code.

Merge pull request #3395 from drwells/pvs-timestep-uninitialized-warning

Fix an member initialization PVS studio warning.

Update memory consumption output.

Apparently, #3333 adds 96 bytes to the memory required by DataOut.

Fix BlockVectorBase::memory_consumption.

The existing implementation forgot to add the memory required for
the std::vector of blocks, it only added the memory required for
the blocks themselves. It turns out that fixing this also allows
to simplify the code significantly.

Make variable names uniform.

Specifically, in DataPostprocessor-derived classes in the tutorial programs,
use the names of function arguments that we also use in DataPostprocessor.

Merge pull request #3430 from bangerth/rename-local-variables

Rename local variables.

Rename local variables.

Instead of uh, duh, dduh, use standard names for variables.

Merge pull request #3393 from davydden/mf_mass_n_q_points_1d

add n_q_points_1d template argument to MatrixFreeOperators::MassOperator

Merge pull request #3391 from davydden/mf_base_diag_fixes

fix matrix-free unit tests due to a change to DiagonalMatrix in Base …

Merge pull request #3428 from bangerth/remove-accidental-files

Remove accidentally added files.

Remove accidentally added files.

Fix an member initialization PVS studio warning.

The variable is now initialized with the same logic used in the
'restart' function.

use fe_degree+2 quadrature points in matrix_free/mass_operator_0x

Silence a PVS studio warning about repeated code.

What we had is correct, but this patch should fix the warning (and make
a little less work for the optimizer).

add n_q_points_1d template argument to MatrixFreeOperators::MassOperator

the rationale is support different usage cases. Say, when projecting
a function one would use fe_degree+2. However when projecting the
MatrixFree quadrature data given by Table<>, most often this data
corresponds to fe_degree+1. So while being given a MatrixFree data
object, we use it even though the mass matrix could be slightly
under-integrated.

fix matrix-free unit tests due to a change to DiagonalMatrix in Base class

while here, also use LaplaceOperator and add a few missing typedefs.

improve error output

Merge pull request #3387 from bangerth/rename-variables

Rename local variables.

Merge pull request #3385 from bangerth/initialize-variable-2

Initialize a member variable.

Merge pull request #3384 from bangerth/initialize-variable

Initialize a member variable.

Merge pull request #3390 from bangerth/fix-invalid-sizeof

Fix a nonsensical use of sizeof().

Fix a nonsensical use of sizeof().

Merge pull request #3386 from bangerth/initialize-variable-3

Initialize a member variable.

Rename local variables.

The variables in question have the same name as a function argument, and
are only used in a local block. They may as well be renamed.

Initialize a member variable.

Also add a piece to the documentation of a member function.

Initialize a member variable.

Initialize a member variable.

Merge pull request #3383 from bangerth/remove-unused-variable

Remove SolverCG::res2.

Merge pull request #3381 from bangerth/remove-duplicate-variable

Remove a duplicate variable.

Merge pull request #3333 from bangerth/update-DataPostprocessor-inputs

Update the way DataPostprocessor receives its input.

Add changelog entry.

Add a bunch of tests.

Update the way DataPostprocessor receives its input.

In the past, the two functions we use to postprocess data got lists of arguments
for the solution, its derivatives, the evaluation points, and normal vectors.
This is not flexible enough: We can not easily add other information that
we have needed in the past or that users have requested, such as a pointer
to the cell we're currently on, or the material-id of the cell.

Rather than adding each possible argument anyone may want to use
individually to the list of the postprocessor function arguments, we use
a technique here that has proven useful in ASPECT: the functions simply
take a reference to a structure that contains these individual pieces
of information. We can extend the members of these structures without
backward compatibility issues because the functions still get a
reference to the same structure, we just grow the structure
itself. Functions that never used the new members of the structure
will continue to work as they always did.

Remove SolverCG::res2.

This variable was only used inside the SolverCG::criterion() function that was
supposed to let derived classes overload when the solver terminates. But the
criterion() function was never called anywhere, so there is no point in
keeping it. Furthermore, the res2 variable it returns was never set. So
remove the whole shebang.

Merge pull request #3382 from bangerth/initialize-member-variable

Initialize all member variables in a copy constructor.

Initialize all member variables in a copy constructor.

Remove a duplicate variable.

The variable already exists in the base class Function. We don't need to
duplicate it in the class FEFieldFunction.

Merge pull request #3341 from davydden/diagonal_matrix

instantiate ConstraintMatrix::distribute_local_to_global() for Diagon…

Merge pull request #3337 from kalj/cuda-default-off

Don't load cuda unless explicitly enabled

instantiate ConstraintMatrix::distribute_local_to_global() for DiagonalMatrix

To that end, add a few missing functions to DiagonalMatrix and update
documentation.

Merge pull request #3340 from davydden/mf_base_updates

use DiagonalMatrix in MatrixFreeOperators

use DiagonalMatrix in MatrixFreeOperators::Base

To that end add a few missing function to DiagonalMatrix and also
switch to Jacobi preconditioner in VectorTools::project() from
quadrature points.

set default compute capability to 3.5, or check that something valid was specified