[sagercaster] Configuration option `observation-period` is ambigeous

[lsiepel]

This parameter is declared as :
<parameter name="observation-period" type="integer" min="0" max="6" required="true">

It is documented as:
observationDescription = Sager WeatherCaster needs a minimum representative period of time to produce meaningfull results (defaults to 6 hours).

So the default of 6 is also the max. What happens when set to 0. The bindings does not seem to handle this case.

I think it should be:
<parameter name="observation-period" type="integer" min="1" required="true">

[lsiepel]

fyi @clinique i found this during extensive checks on all documentation and metadata files of all bindings.

[clinique]

You're correct, the max value is pointless. I would keep min to 0 - this is weak from the algorithm point of view but eases immediate feed-back for the user when trying it.
But BTW this gives me the idea that the operation period could grow dynamically up to 6 hours after binding start.

[clinique]

Closed by #19574