Skip to content
This repository was archived by the owner on Jan 24, 2024. It is now read-only.

Conversation

@SammyRamone
Copy link
Collaborator

Proposed changes

This way foot is really just the strain gauges. this way the footsensors are working better

Related issues

Necessary checks

  • Update package version
  • Run catkin build
  • Write documentation
  • Create issues for future work
  • Test on your machine
  • Test on the robot
  • Put the PR on our Project board

@SammyRamone SammyRamone self-assigned this Oct 26, 2022
@jaagut
Copy link
Member

jaagut commented Feb 14, 2023

@SammyRamone @jgueldenstein is this still relevant? Do we want to merge this?

@SammyRamone
Copy link
Collaborator Author

I think it should not be merged like this. But the underlying issue should be addressed. The collision model of the foot itself is always making contact to the ground due to the physics of the ground which let it "sink" into the ground.

@jaagut jaagut marked this pull request as draft August 9, 2023 14:21
@jaagut
Copy link
Member

jaagut commented Aug 9, 2023

@jgueldenstein Still relevant?

@SammyRamone
Copy link
Collaborator Author

@jgueldenstein Still relevant?

I'm not Jasper, but I will answer this. I'm not sure if it is still relevant. Check the output of the simulated strain gauges and click on the robot to see the active bounding boxes. The problem was that not only the strain gauges were colliding with the floor (due to the robot sinking into the "soft" grass) and therefore the values of the sensors were very wrong. This PR is more of a hacky fix for this as it removes the center collision part of the foot. There might be a better solution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants