Skip to content

Feature/analytical ik#781

Merged
jaagut merged 26 commits intomainfrom
feature/analytical_ik
Apr 11, 2026
Merged

Feature/analytical ik#781
jaagut merged 26 commits intomainfrom
feature/analytical_ik

Conversation

@jaagut
Copy link
Copy Markdown
Member

@jaagut jaagut commented Mar 13, 2026

Summary

Proposed changes

Related issues

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

Flova added 4 commits March 13, 2026 15:20
Signed-off-by: Florian Vahl <florian@flova.de>
Signed-off-by: Florian Vahl <florian@flova.de>
Signed-off-by: Florian Vahl <florian@flova.de>
Signed-off-by: Florian Vahl <florian@flova.de>
@github-project-automation github-project-automation bot moved this to 🆕 New in Software Mar 13, 2026
@jaagut jaagut moved this from 🆕 New to 🏗 In progress in Software Mar 13, 2026
@jaagut jaagut assigned jaagut and Flova and unassigned jaagut Mar 13, 2026
Copy link
Copy Markdown
Member Author

@jaagut jaagut left a comment

Choose a reason for hiding this comment

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

My review from yesterday...

bool success;
goal_state_->updateLinkTransforms();

// Kann man den einfach löschen? Scheint ja nciht so benutzt zu werden
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

???

Comment thread src/bitbots_motion/bitbots_dynup/include/bitbots_dynup/ik.hpp
@jaagut
Copy link
Copy Markdown
Member Author

jaagut commented Mar 16, 2026

CI will fail because of our server downtime

@jaagut jaagut requested a review from Flova March 16, 2026 18:16
@jaagut jaagut moved this from 🏗 In progress to 👀 In review in Software Mar 16, 2026
@Flova
Copy link
Copy Markdown
Member

Flova commented Mar 21, 2026

Do we want to merge this? Maybe just to add the go 2026 tag

@jaagut
Copy link
Copy Markdown
Member Author

jaagut commented Mar 21, 2026

I mean, from the software side, this is a good and working state for Wolfgang. Worthy of a GO Tag and a renewed Wolfgang Tag.

@jaagut jaagut merged commit 5f81e12 into main Apr 11, 2026
2 checks passed
@jaagut jaagut deleted the feature/analytical_ik branch April 11, 2026 12:59
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Software Apr 11, 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.

5 participants