change default LatentDelay constructor to right truncation at 99th perc

[SamuelBrand1]

I spotted that the default behaviour of the usual LatentDelay constructor was to have right truncation at 15 time steps irrespective of the underlying delay distribution. This is obviously a potential source of making silent errors.

This PR changes the constructor so that default of behaviour of this constructor is to calculate the 99th percentile of the observation distribution i.e.

$$x_{99} = F^{(-1)}(0.99).$$

Where $F^{(-1)}$ is the inverse of the distribution function of the delay¹. And use the rounded value of $x_{99}$ as the right truncation value for the constructor.

Footnotes

1. NB: the constructor is only defined for continuous delay distributions so this is always valid. ↩

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.34%. Comparing base (e73fd9e) to head (74c818a).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #265      +/-   ##
==========================================
+ Coverage   93.31%   93.34%   +0.02%     
==========================================
  Files          45       45              
  Lines         449      451       +2     
==========================================
+ Hits          419      421       +2     
  Misses         30       30              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[seabbs]

Nice. LGTM