🧹 fix typo in docstring + remove LTTB from MinMaxLTTB + remove interleave_gaps

[jonasvdd]

No description provided.

[codecov-commenter]

Codecov Report

Merging #201 (a43fa54) into main (614c9de) will decrease coverage by 0.33%.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main     #201      +/-   ##
==========================================
- Coverage   97.44%   97.11%   -0.33%     
==========================================
  Files          13       13              
  Lines         979      972       -7     
==========================================
- Hits          954      944      -10     
- Misses         25       28       +3     

Impacted Files	Coverage Δ
...tly_resampler/aggregation/aggregation_interface.py	94.91% <ø> (-0.09%)	⬇️
...tly_resampler/figure_resampler/figure_resampler.py	89.16% <ø> (ø)
...ler/figure_resampler/figure_resampler_interface.py	100.00% <ø> (ø)
plotly_resampler/aggregation/aggregators.py	100.00% <100.00%> (ø)
plotly_resampler/aggregation/gap_handlers.py	96.00% <100.00%> (-4.00%)	⬇️
..._resampler/aggregation/plotly_aggregator_parser.py	94.44% <100.00%> (-2.19%)	⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[jvdd]

still 1 job that failed over the nans...

[jonasvdd]

Not consistently, I guess? @jvdd

[jayceslesar]

I arrived at almost the same number when I implemented this..funny how that works

[jvdd]

Still seems to fail here and there (but not consistently), but already significantly fewer test fails than with 4 🙃

[jonasvdd]

@jvdd, Do I handle here the Interleave gaps issue as well?

[jvdd]

@jvdd, Do I handle here the Interleave gaps issue as well?

@jonasvdd sure!

[jonasvdd]

Okay @jvdd, decided to open a seperate PR which will be merged into this one w.r.t. the interleave gaps, see #204

[jonasvdd]

I think this PR can be merged as well @jvdd?

[jvdd]

LGTM!!

Still not entirely sure whether 4.1 is a perfect multiplier for our approximated med_diff.. Nonetheless, this (:cherries: :pick:) value seems to be an improvement over the 4 that was used before 🪄

Let's merge @jonasvdd 🤝