Hello,
Amazing work! Keep it going
I found one problem with trashing. I was experimenting with fishing villages, and noticed bot wants to leave at least 3 coppers before trashing them. And when counting coppers it does not include fishing willage's +1 coin. So it leads to situations, where it thashes less than it should to. I copy/pasted one example. Here bot has already bought fishing village, so it needs two coppers to keep buying power, but it keeps three of them. I guess trashing algorithm is hard coded, but maybe i am wrong? Here is log:
== BankWharf's turn 1 ==
BankWharf plays Copper.
BankWharf plays Copper.
BankWharf plays Copper.
Coins: 3, Potions: 0, Buys: 1
BankWharf buys Chapel.BankWharf draws 5 cards (Copper,Estate,Copper,Copper,Copper).
== BankWharf's turn 2 ==
BankWharf plays Copper.
BankWharf plays Copper.
BankWharf plays Copper.
BankWharf plays Copper.
Coins: 4, Potions: 0, Buys: 1
BankWharf buys Fishing Village.(BankWharf shuffles.)
BankWharf draws 5 cards (Estate,Copper,Copper,Chapel,Fishing Village).
== BankWharf's turn 3 ==
BankWharf plays Fishing Village.
BankWharf plays Chapel.
BankWharf trashes Estate.
BankWharf
trashes Copper.
BankWharf
trashes Copper.
Coins: 1, Potions: 0, Buys: 1
BankWharf draws 5 cards (Copper,Copper,Copper,Estate,Copper).
== BankWharf's turn 4 ==
BankWharf resolves the duration effect of Fishing Village.
BankWharf plays Copper.
BankWharf plays Copper.
BankWharf plays Copper.
BankWharf plays Copper.
Coins: 5, Potions: 0, Buys: 1
BankWharf buys Wharf.
(BankWharf shuffles.)
BankWharf draws 5 cards (Estate,Copper,Chapel,Copper,Copper).
== BankWharf's turn 5 ==
BankWharf plays Chapel.
BankWharf trashes Estate.
BankWharf
trashes Copper.BankWharf
trashes Copper.BankWharf
plays Copper. -
should trash it too, because bot still is capable of buying silver with 2 coppers ant fishing villageCoins: 1, Potions: 0, Buys: 1
BankWharf draws 5 cards (Estate,Copper,Wharf,Fishing Village,Copper).
Good luck