fslaborg / FSharp.Stats

statistical testing, linear algebra, machine learning, fitting and signal processing in F#
https://fslab.org/FSharp.Stats/
Other
205 stars 54 forks source link

Add merge strategies for maps #264

Closed bvenn closed 1 year ago

bvenn commented 1 year ago

Thank you for contributing to FSharp.Stats. Please take the time to tell us a bit more about your PR.

Please reference the issue(s) this PR is related to

263

Please list the changes introduced in this PR

Description

Maps are the foundation of modules Frequency and Distribution.Empirical. Frequency maps can be merged of data histograms should be combined. Generic mergeBy functions enable to decide what happens when keys are present in both input maps.

[Required] please make sure you checked that

[Optional]

codecov-commenter commented 1 year ago

Codecov Report

Merging #264 (814ae18) into developer (3d6a220) will increase coverage by 0.08%. The diff coverage is 61.22%.

:mega: 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              @@
##           developer     #264      +/-   ##
=============================================
+ Coverage      46.41%   46.49%   +0.08%     
=============================================
  Files            152      153       +1     
  Lines          15255    15304      +49     
  Branches        1980     1983       +3     
=============================================
+ Hits            7080     7115      +35     
- Misses          7628     7641      +13     
- Partials         547      548       +1     
Impacted Files Coverage Δ
src/FSharp.Stats/Distributions/Frequency.fs 0.00% <0.00%> (ø)
src/FSharp.Stats/Distributions/Empirical.fs 24.41% <33.33%> (+0.96%) :arrow_up:
src/FSharp.Stats/Map.fs 66.66% <66.66%> (ø)
tests/FSharp.Stats.Tests/DistributionsEmpirical.fs 100.00% <100.00%> (ø)