octachrome / treason

A clone of the card game Coup written in Node.js
Other
139 stars 79 forks source link

AIs take income even when 3 dukes are out #25

Closed Hypak closed 4 years ago

Hypak commented 5 years ago

When 3 dukes are revealed, the AI takes income instead of foreign aid despite the fact that foreign aid is perfectly safe to take. (Edit: I've realized this is similar to #21, however the AIs can be sure that foreign aid is safe to take in this scenario) (Edit 2: I've now realized that there was an attempt to fix this issue https://github.com/octachrome/treason/pull/18. I'll try to get a screenshot of the AIs acting in this way)

Hypak commented 5 years ago

screen shot 2019-02-16 at 12 01 48

octachrome commented 5 years ago

Thanks for the issue report. Sorry I haven't replied earlier, I've not had any time for this project recently. It is odd that this issue still exists given the fix for #18.

NobleUplift commented 4 years ago

According to version.js, the site is still running v0.1.8-160-gfed5832, was the fixed pushed to production?

octachrome commented 4 years ago

Good point! This is now pushed.