Inference engine - Add new bridge bidding bot. #216
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Various fixes to Jacoby Transfers and new test files. Basic transfer logic working.
All transfer tests work properly. Now need to implement Stayman as a state machine.
More tests. All NT conventions now using state machine. Still needs lots of cleanuo
More cases are working. Now have 2NT tests. Stayman and transfers are working properly for basic cases. All using state machine call-backs. Lots remains to be done - interference needs to be considered at each step.
Also implemented basic 1NT overcall test
Added new test for Stayman. Cleaned up Stayman.cs. Moved non-conventional bids into NoTrump class
Removed ranges from NT and moved to an enum that describes various ranges. First attempt at slam bids.
Now 5/4 invitational hands work per SAYC bidding. Additional regression tests pass.
Was using old ranges for points. Changed to Non-Forcing bid function.
Cappelletti is not part of SAYC. Did not remove file, but removed all calls to it. Added new test for simple NT overcalls
Added PointsAdjustment call to bids. Dummy computation requires a suit.
All code should now be in NoTrump.cs.
Prototype code for a new way to represent bid rules as a series of constraints. The rules can update a global summary state, and examine that summary state in subsequent bids.
Still a major work in progress, but huge changes so I an committing.
Still lots to do, and hack implementaitons, but basic bidding is actually working now.
The most basic of basic tests are working.
Fixed several bugs. Implemented contract. Bid validation over a contract. JumpBid constraint.
Most stayman tests now work. PBN tests plugged into hacked test infrastructue.
Did a lot of cleanup on NT conventions. Have HCP as its own value in HandSummary. Can use StartingPoints and HighCardPoints together to make decisions
Still more cleanup - lots to do. But get it on my desktop...
Still lots of cleanup to do but more basic bidding tests passing.
…System interface.
Now its just a matter of making them all work...
Almost everything except for doubles (both penalty and takeout) is working correctly.
Hacked to let webui work with new bot
And use RLBot by default in test suite.
…ricksterBots into InferenceEngine
Update inference engine to run conditionally
|
@RalphLipe let me know once you've finished removing the remaining dependencies on TricksterBotClasses. I've put together a test project using .NET Standard 2.0 (.NET 8 for tests) that I'll help migrate this to. That'll enable folks to test/contribute from other OSes like Macs. I'll also move everything to a separate repository at that point so we can have dedicated issue/code management for this project apart from the other bots. |
|
I am about 1/3 of the way through removing dependencies.
…On Fri, Dec 8, 2023 at 10:13 AM Tony Ross ***@***.***> wrote:
@RalphLipe <https://github.com/RalphLipe> let me know once you've
finished removing the remaining dependencies on TricksterBotClasses.
I've put together a test project using .NET Standard 2.0 (.NET 8 for
tests) that I'll help migrate this to. That'll enable folks to
test/contribute from other OSes like Macs. I'll also move everything to a
separate repository at that point so we can have dedicated issue/code
management for this project apart from the other bots.
—
Reply to this email directly, view it on GitHub
<#216 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADAT5HLPDNMFA74A324NICDYINDD5AVCNFSM6AAAAABABZFYEGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBXGU2DKNBRGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Introduced the BridgeBidding namespace and the BridgeBidder class. All dependencies on Trickster classes have been removed. Now the bidding bot can be moved to a completely independent project.
Check to see if opening bid was No Trump before applying logic for takeout double.
One more test passes, and a new one breaks. Not 100% sure if new break is really an incorrect test. Blackwood advance now works properly though.
|
This has been moved to our Bridgit repository. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.