Skip to content

Keep a calm head#2084

Open
oleflb wants to merge 2 commits intoHULKs:mainfrom
oleflb:keep-a-calm-head
Open

Keep a calm head#2084
oleflb wants to merge 2 commits intoHULKs:mainfrom
oleflb:keep-a-calm-head

Conversation

@oleflb
Copy link
Copy Markdown
Contributor

@oleflb oleflb commented Jul 20, 2025

Why? What?

Use lowpass filter for all head movements.

Fixes #2052

ToDo / Known Issues

  • Remove velocity limit code

Ideas for Next Iterations (Not This PR)

If there are some improvements that could be done in a next iteration, describe them here.

How to Test

Describe how to test your changes. (For the reviewer)

@github-project-automation github-project-automation Bot moved this to Request for Review in Development Jul 20, 2025
pub outer_minimum_pitch: f32,
pub outer_yaw: f32,
pub injected_head_joints: Option<HeadJoints<f32>>,
pub filter_smoothing_factor: f32,
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.

Suggested change
pub filter_smoothing_factor: f32,
pub smoothing_factor: f32,

let positions = request.positions;
let last_positions = self.last_request.positions;

let velocity_limited_positions = HeadJoints {
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.

Please remove velocity limiting

@schluis
Copy link
Copy Markdown
Contributor

schluis commented Jul 20, 2025

Hammerding 🥵

@schluis schluis self-assigned this Jul 21, 2025
@pejotejo pejotejo moved this from Request for Review to NaoEra Backlog in Development Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: NaoEra Backlog

Development

Successfully merging this pull request may close these issues.

Bottom Camera Destabilizes Walking

3 participants