Am 18.06.2018 um 17:03 schrieb Riccardo
(Jack) Lucchetti:
On
Mon, 18 Jun 2018, Allin Cottrell wrote:
On Mon, 18 Jun 2018, Riccardo (Jack)
Lucchetti wrote:
On Mon, 18 Jun 2018, Sven Schreiber
wrote:
Instead I think this inconsistent
input should throw an error.
The way the code works as of now is that leave-one-out can be
triggered by a negative bandwidth OR by the Boolean fourth
argument. Perhaps you're right, we should raise an error in
the case of inconsistency. Other opinions?
I think under the new scheme a negative bandwidth should not be
accepted when more than three arguments are given. For backward
compatibility it could still be accepted, with its original
meaning, whhen just three arguments are given.
Makes sense. Now in git.
OK. Then a related question is how to deal with the nadarwat_trim
setting when the fifth (trimming) argument is given. My suggestion
would be to print a deprecation warning in the case where (i) the
nadarwat_trim setting has been user-adjusted (to a non-default
value) and (ii) a different value is given as the fifth arg.
But I admit that maybe it's too much of a fringe issue.
thanks,
sven