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.

Sign up for GitHub

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

Jump to bottom

Give AI players a budget to spend in auctions #803

Merged
merged 1 commit into from
Oct 14, 2024
Merged

Give AI players a budget to spend in auctions #803

merged 1 commit into from
Oct 14, 2024

Conversation

toberge
Copy link
Contributor

@toberge toberge commented Jul 14, 2024 •
edited
Loading

  • define simple budget
  • make it depend on gamemode (only in first to 30 it takes effect)
  • make bots yield when they've reached budget
  • and prevent them from bidding on things that would take them beyond budget (unless especially good I suppose?)

Closes #863

@toberge toberge marked this pull request as draft July 14, 2024 17:05
@toberge toberge force-pushed the feature/bidding-ai-budget branch 2 times, most recently from cfa7067 to 28955cb Compare October 10, 2024 21:41
@toberge toberge marked this pull request as ready for review October 11, 2024 19:13
@toberge toberge force-pushed the feature/bidding-ai-budget branch 2 times, most recently from 963897c to 246063d Compare October 11, 2024 19:51
Fueredoriku
Fueredoriku approved these changes Oct 13, 2024
View reviewed changes
Copy link
Contributor

@Fueredoriku Fueredoriku left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Nice improvement of the first to 30 AIs!

It would be nice if they additionaly didn't bid on augments they already own too, but that could be done in a later PR, so I'll leave an approve for you to use at your discretion 👍

@toberge toberge merged commit 5fa0e00 into dev Oct 14, 2024
3 checks passed
@toberge toberge deleted the feature/bidding-ai-budget branch October 14, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fueredoriku Fueredoriku Fueredoriku approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Prosjekt-Spill-H22 board
Status: Done 🎉
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Limit AI player spending in auctions
2 participants
@toberge @Fueredoriku