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

simple max advance #13

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

cortlandstarrett
Copy link
Member

This is a minimalist change that has a pleasingly visible effect on the behavior of the bot. The point of this change is primarily to serve as an example of a small change. Secondarily, it is kinda cool to see a simplistic but quite aggressive behavior.

  1. create new package 'move_selection'
  2. added function 'simple_max_advance' taking as input the array
    of legal moves and our own color (Color::WHITE or Color::BLACK).
  3. 'simple_max_advance' selects the move that increases our rank
    (or decreases if BLACK) by the most. When no advance is possible,
    select a random move.
  4. Change 'send_challenge' to send a challenge to my other bot
    which has been running the base teacher model (random selection).

The following link shows this strategy in action running as BLACK.

https://lichess.org/iHC7cRqI

This is a minimalist change that has a pleasingly visible effect
on the behavior of the bot.  The point of this change is primarily
to serve as an example of a small change.  Secondarily, it is
kinda cool to see a simplistic but quite aggressive behavior.

1) create new package 'move_selection'
2) added function 'simple_max_advance' taking as input the array
   of legal moves and our own color (Color::WHITE or Color::BLACK).
3) 'simple_max_advance' selects the move that increases our rank
   (or decreases if BLACK) by the most.  When no advance is possible,
   select a random move.
4) Change 'send_challenge' to send a challenge to my other bot
   which has been running the base teacher model (random selection).

The following link shows this strategy in action running as BLACK.

https://lichess.org/iHC7cRqI
In the main branch, we have settled on using a separate move count
rather than a length operator on the array, which is not supported
in MC-3020.
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.

1 participant