Skip to content

NF: Introduce food_eaten counter - #968

Open
Debilski wants to merge 2 commits into
ASPP:mainfrom
Debilski:feature/food_eaten-flag
Open

NF: Introduce food_eaten counter#968
Debilski wants to merge 2 commits into
ASPP:mainfrom
Debilski:feature/food_eaten-flag

Conversation

@Debilski

@Debilski Debilski commented Jul 8, 2026

Copy link
Copy Markdown
Member

Adds a new counter gamestate["food_eaten"] and bot.food_eaten that tracks the number of food pellets eaten by a specific bot.

image

If there is consensus, I would also remove the deaths from the UI as they are identical to the number of kills of the other team.

Closes #911

@Debilski
Debilski force-pushed the feature/food_eaten-flag branch from e19fe25 to 9384ee4 Compare July 8, 2026 14:23
@Debilski
Debilski force-pushed the feature/food_eaten-flag branch from 9384ee4 to d5ff18a Compare July 8, 2026 14:33
@otizonaizit

Copy link
Copy Markdown
Member

Is this a team property or a bot property?
I'm not sure what is this useful for?

Isn't this 30 - len(bot.enemy.food)?

If it is just for display in the UI, maybe we don't need to expose it in the bot object?

In the GUI I would call it simply food instead of food eaten.

Agreed on removing deaths

@Debilski

Debilski commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

It is on a per-bot level so it is a bit more accurate than 30 - len(bot.enemy.food). Not sure if it is needed but if we have bot.kills then we might as well include this.

@Debilski
Debilski force-pushed the feature/food_eaten-flag branch from 74670db to 0c9ef7e Compare July 8, 2026 14:47
@otizonaizit

otizonaizit commented Jul 8, 2026

Copy link
Copy Markdown
Member

It is on a per-bot level so it is a bit more accurate than 30 - len(bot.enemy.food). Not sure if it is needed but if we have bot.kills then we might as well include this.

Wait, so bot.kills is a bot-level counter, not a team-level counter? Why did we do this?

@Debilski

Debilski commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

b351160 🤷

@Debilski

Debilski commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

I guess we can also postpone the merging of this until we have decided whether we want to thin out the Bot attributes.

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.

Remove number of deaths from UI (and maybe add food eaten?)

2 participants