oakmound / oak

A pure Go game engine
Apache License 2.0
1.52k stars 84 forks source link

alg\selection: WeightedMapChoice previously did not appropriately use cumulative weights #195

Closed Implausiblyfun closed 2 years ago

Implausiblyfun commented 2 years ago

One part (there may be more) for https://github.com/oakmound/oak/issues/179

codecov-commenter commented 2 years ago

Codecov Report

Merging #195 (0180c8b) into master (1db3119) will decrease coverage by 0.09%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #195      +/-   ##
==========================================
- Coverage   92.83%   92.74%   -0.10%     
==========================================
  Files         141      141              
  Lines        6406     6407       +1     
==========================================
- Hits         5947     5942       -5     
- Misses        399      405       +6     
  Partials       60       60              
Impacted Files Coverage Δ
alg/selection.go 100.00% <100.00%> (ø)
drawLoop.go 62.26% <0.00%> (-11.33%) :arrow_down:
event/handler.go 96.66% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more