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