Commit 9840890e authored by Martin Reinecke's avatar Martin Reinecke
Browse files

Merge remote-tracking branch 'origin/NIFTy_5' into documentation_feedback

parents 67a2d129 9f8bd423
Pipeline #45147 passed with stages
in 9 minutes and 25 seconds
......@@ -35,6 +35,19 @@ class ScalingOperator(EndomorphicOperator):
:class:`Operator` supports the multiplication with a scalar. So one does
not need instantiate :class:`ScalingOperator` explicitly in most cases.
Formally, this operator always supports all operation modes (times,
adjoint_times, inverse_times and inverse_adjoint_times), even if `factor`
is 0 or infinity. It is the user's responsibility to apply the operator
only in appropriate ways (e.g. call inverse_times only if `factor` is
Along with this behaviour comes the feature that it is possible to draw an
inverse sample from a :class:`ScalingOperator` (which is a zero-field).
This occurs if one draws an inverse sample of a positive definite sum of
two operators each of which are only positive semi-definite. However, it
is unclear whether this beviour does not lead to unwanted effects
somewhere else.
def __init__(self, factor, domain):
......@@ -44,10 +57,7 @@ class ScalingOperator(EndomorphicOperator):
raise TypeError("Scalar required")
self._factor = factor
self._domain = makeDomain(domain)
if self._factor == 0.:
self._capability = self.TIMES | self.ADJOINT_TIMES
self._capability = self._all_ops
self._capability = self._all_ops
def apply(self, x, mode):
self._check_input(x, mode)
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment