-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve iterators #80
Conversation
bors r+ |
80: Improve iterators r=charleskawczynski a=charleskawczynski - Button -> Dealer - circle_table -> circle_index, which only takes the length and state for indexing `players` - Remove position - add `n_iter` to iterators, so that we can collect the results and simplify the tests Co-authored-by: Charles Kawczynski <kawczynski.charles@gmail.com>
bors r- |
Canceled. |
ded08e5
to
b010a02
Compare
bors r+ |
Closes #81 |
This doesn't close #73, but we may be able to just skip non-active players on the fly. This PR at least ensures that we start iterating on only active players. |
Codecov Report
@@ Coverage Diff @@
## main #80 +/- ##
==========================================
+ Coverage 86.81% 87.20% +0.39%
==========================================
Files 8 8
Lines 872 899 +27
==========================================
+ Hits 757 784 +27
Misses 115 115
Continue to review full report at Codecov.
|
circle_table
->circle_index
, which only takes the length and state for indexingplayers
position
n_iter
to iterators, so that we can collect the results and simplify the tests