Skip to content

Handle localisation after in-place penalties#888

Merged
cleWu03 merged 7 commits into
mainfrom
in_place_penalties-in-gamestate-und-localisation
Jun 7, 2026
Merged

Handle localisation after in-place penalties#888
cleWu03 merged 7 commits into
mainfrom
in_place_penalties-in-gamestate-und-localisation

Conversation

@cleWu03

@cleWu03 cleWu03 commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Summary

In case of a in place penalty we don't want to reset the localisation to the field size, therefore we check in the gameState message for the penalty type and integrate it in the decisions of the localisation.dsd

Checklist

  • Run pixi run build
  • Write documentation
  • Test on your machine
  • Test on the robot
  • Create issues for future work
  • Triage this PR and label it

@github-project-automation github-project-automation Bot moved this to 🆕 New in Software Jun 6, 2026
@cleWu03 cleWu03 requested review from ayin21 and jaagut June 6, 2026 14:36
@github-project-automation github-project-automation Bot moved this from 🆕 New to 👀 In review in Software Jun 6, 2026
@cleWu03 cleWu03 changed the title add field in gamestate msg and check for it in localisation.dsd Handle localisation after in-place penalties Jun 6, 2026
@cleWu03 cleWu03 merged commit a8db210 into main Jun 7, 2026
2 checks passed
@cleWu03 cleWu03 deleted the in_place_penalties-in-gamestate-und-localisation branch June 7, 2026 12:00
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Software Jun 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants