Skip to content
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

Feature: Add dead cards option for evaluation #82

Closed
wants to merge 1 commit into from

Conversation

LukeLemes
Copy link

@LukeLemes LukeLemes commented Jan 30, 2024

Dead cards option for evaluation in games Holdem, Omaha etc.
Added for enabling multi-board evaluation.

I wanted this feature also on cmdline ... do you think we can merge this? ... It helps evaluate double board bomp pots

Dead cards option for evaluation in games Holdem, Omaha etc.
Add to enable multi-board evaluation.
@andrewprock
Copy link
Owner

This can't be merged because it will break existing code. We need to add a wrapper that preserves the existing signature so that old code will still compile.

@LukeLemes
Copy link
Author

OK ... so the overloading this fce could work, could it not?

@andrewprock
Copy link
Owner

Using a default argument of the empty set, or creating a separate method would work.

@LukeLemes LukeLemes deleted the branch andrewprock:master December 28, 2024 15:28
@LukeLemes LukeLemes closed this Dec 28, 2024
@LukeLemes LukeLemes deleted the master branch December 28, 2024 15:28
@LukeLemes LukeLemes restored the master branch December 28, 2024 15:31
@LukeLemes LukeLemes deleted the master branch January 2, 2025 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants