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

DEV 1410 allow placement of up to 16 snakes on very large boards #90

Merged
merged 1 commit into from
Jun 29, 2022

Conversation

torbensky
Copy link
Collaborator

No description provided.

func isKnownBoardSize(b *BoardState) bool {
func isExtraLargeBoardSize(b *BoardState) bool {
// We can do placement for any square, large board using the distributed placement algorithm
return b.Width == b.Height && b.Width >= 21
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I should restrict this to odd size boards? The placement is a bit wonky when the board size is even.

Copy link
Contributor

@robbles robbles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested out the new algorithm with 9-16 snakes on a few square board sizes above 21x21, and everything looks great!

@robbles robbles merged commit 61aeee3 into main Jun 29, 2022
@robbles robbles deleted the DEV-1410-snake-placements-large-map branch June 29, 2022 21:26
coreyja added a commit to coreyja/rules that referenced this pull request Jul 2, 2022
* main:
  prevent negative hazard damage from healing past max health (BattlesnakeOfficial#91)
  allow placement of up to 16 snakes on xlarge board (BattlesnakeOfficial#90)
  update example output from battlesnake play --help
  DEV 1404: Support streaming CLI games to the browser board (BattlesnakeOfficial#88)
  speeding up how fast the expanding box grows from every 15 to every 12 turns (BattlesnakeOfficial#89)
  fix for rivers and bridges snake start positions (BattlesnakeOfficial#85)
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