search

skent259 / crapssim

Simulator for craps with various betting strategies
MIT License
28 stars 13 forks source link

Improve Table Payouts per issue #13 #18

Closed amortization closed 2 years ago

amortization commented 2 years ago

See Issue #13 for discussion.

This changes Table.payouts to store all bet information that varies by table on the Table (ex. Field Payouts, Fire Bet Point payouts, etc.) It also makes table an attribute of Bet in order to access those variables when doing the update logic. It also makes Bet.payout_ratio a property of bet instead of an attribute so for bets where the payout ratio varies based on table settings or other logic than a static number.

amortization commented 2 years ago

@skent259 for some reason there was two of each of your comments, so I deleted one of each and replied to the others.

skent259 commented 2 years ago

Sorry about the dual comments, not sure why that happened.

I'm happy to merge this and follow-up the requested changes in #19, given that my comments were all minor and there's a lot of good work in this to incorporate. @amortization let me know if that works for you