Open amortization opened 2 years ago
Good catch, that's most certainly a bug. I think this strategy was written before the place bet logic was fully understood. The small problem is that I've talked about this strategy previously: https://pages.stat.wisc.edu/~kent/blog/2019.07.31_Craps_Budget/craps_best-strategies-on-a-budget.html
The question is whether this is a reasonably useful strategy for someone or not? The logic is kind of complicated, only putting new place bets up when both win or both lose.
In the new strategy syntax, a proper place 68 strategy is easily combined with BetPassLine(5) + BetPlace({6: 5, 8:5}), right? So I don't think there needs to be a formal definition of this strategy.
My inclination is to implement it, as is, but with a slightly different name. Perhaps BetPassPlace68Jointly
? That way there's not really a breaking change but it's clear that this isn't the typical way the strategy would be implemented.
Thoughts?
Yeah, I think that leaving it but renaming it to something more clear would make the most sense.
I'm working on creating integration tests for #29 and a question came up.
What I'm doing is running a script to produce data from the Master branch, and then making sure that the data in the #29 branch matches what was output from master for each strategy. It is working through the passline and passlineodds strategies, but is running into issues with place68.
The original code for place68 was
So what is happening in the original script is that if a player has a Place6 or Place8 bet that wins and gets taken down the Place6 or Place8 doesn't get left up. For example:
after running table.add_player_bets() what was originally left up was:
What I would normally expect would be that the Place8 bet gets re-added (since normally players don't take down place bets after winning them, the dealer just gives them their winnings) so the bets_after would be:
I think what is probably happening is that it's hitting this logic
and seeing that the player has a Place6 and not ever placing the Place8 back up. It currently would wait until the player won the Place6, and then it would put both of those bets back up. I think that the more natural expectation for this strategy would be to leave the Place8 up in this case and just take the winnings since when most people talk about placing the 6 and 8 that's what they are referring to, but it would mean that the new implementation of this strategy would differ from the original. Thoughts on this?