Skip to content

Fix/parameters for positioning capsule - #1079

Open
cleWu03 wants to merge 5 commits into
mainfrom
fix/parameters_for_positioning_capsule
Open

cleWu03 wants to merge 5 commits into
mainfrom
fix/parameters_for_positioning_capsule

Conversation

@cleWu03

@cleWu03 cleWu03 commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Rename unclear variables
  • Some updates on variable descriptions
  • Check for absolute or relative values
  • All params included into the body_behavior.yaml

Part of #1056 but doesnt include all the there suggested features. Would be nice to merge it soon anyway to enable working on the positioning capsule without merge conflicts in the future

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

@MegaIng MegaIng left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The parameters in body_behavior.yaml should be moved into a block "formation" or "positioning" so that it's clear that they belong together. This then also means "formation_include_supporter".

Also, the list of sliders and the way those values are being read out and put into the params in the debug script should be refactored to be maintainable. It's a large unmaintainable script but by tackling sections as we have to touch them we will get there ;-)

@github-project-automation github-project-automation Bot moved this from 🆕 New to 🏗 In progress in Software Sep 20, 2026
…gui sliders and rename gui internal variables
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🏗 In progress

Development

Successfully merging this pull request may close these issues.

2 participants