codecov[bot]
commented
3 years ago Codecov Report
Merging #5 (4777b74) into main (ec520a0) will decrease coverage by
0.26%. The diff coverage is45.16%.
@@ Coverage Diff @@
## main #5 +/- ##
==========================================
- Coverage 48.14% 47.88% -0.27%
==========================================
Files 1 2 +1
Lines 81 71 -10
==========================================
- Hits 39 34 -5
+ Misses 42 37 -5 | Impacted Files | Coverage Δ | |
|---|---|---|
| src/hand_eval.jl | 27.65% <27.65%> (ø) |
|
| src/HoldemHandEvaluator.jl | 87.50% <100.00%> (+39.35%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update ec520a0...4777b74. Read the comment docs.
charleskawczynski
bors[bot]
This PR
prime, defined for each rank and uses the product of primes as a unique key for all combinations to map to the method lookup (described here).Rather than using one long vector, I thought it might be interested to still generate methods so that this can remain gpu-compatible.
Closes #3.