Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use foot sensor to detect steps / rework pressure converter. #337

Open
wants to merge 40 commits into
base: main
Choose a base branch
from

Conversation

val-ba
Copy link
Contributor

@val-ba val-ba commented Feb 2, 2024

Summary

We do not detect steps.

Proposed changes

We can use the foot pressure sensors to detect them. The odometry gets more accurate if we know that.

Related issues

Checklist

  • Run colcon build
  • Write documentation
  • Create issues for future work
  • Test on your machine
  • Test on the robot
  • This PR is on our Software project board

@val-ba val-ba self-assigned this Feb 2, 2024
@val-ba val-ba added the enhancement New feature or request label Feb 2, 2024
@Flova Flova added the motion label Feb 2, 2024
@jaagut jaagut changed the title Use foot pressure sensors to detect steps Use ~~foot pressure~~ IMU sensors to detect steps Jun 5, 2024
@jaagut jaagut changed the title Use ~~foot pressure~~ IMU sensors to detect steps Use ̶f̶o̶o̶t̶ ̶p̶r̶e̶s̶s̶u̶r̶e̶ IMU sensor to detect steps Jun 5, 2024
@jaagut
Copy link
Member

jaagut commented Jun 5, 2024

It seems like, we have not the latest changes from yesterday in here, right @Flova ?

@Flova
Copy link
Member

Flova commented Jun 5, 2024

The things from yesterday are a completely different change. Please rename undo the renaming ^^

This PR tried to use the foot sensors in general. It reworks the pressure converter and adds the footstep detector. For now detecting footsteps with the foot sensors does not seem possible, but I would maybe try to merge the pressure converter and keep this changes somewhere in case we get better foot pressure sensors and are able to utilize them.

@Flova Flova changed the title Use ̶f̶o̶o̶t̶ ̶p̶r̶e̶s̶s̶u̶r̶e̶ IMU sensor to detect steps Use foot sensor to detect steps / rework pressure converter. Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request motion
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

4 participants