Skip to content

Added Hydrostatic Pressure sensor#3497

Open
ahcorde wants to merge 1 commit intomainfrom
ahcorde/main/hydrostatic_pressure
Open

Added Hydrostatic Pressure sensor#3497
ahcorde wants to merge 1 commit intomainfrom
ahcorde/main/hydrostatic_pressure

Conversation

@ahcorde
Copy link
Copy Markdown
Contributor

@ahcorde ahcorde commented Apr 29, 2026

🎉 New feature

Summary

I noticed that there is not Hydrostatic Pressure sensor which publish a fluidPressure message. We have Altimeter or Air Pressure.

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the feature
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Generated-by: Claude Sonnet 4.6

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Backports: If this is a backport, please use Rebase and Merge instead.

Signed-off-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from caguero April 29, 2026 09:44
@ahcorde ahcorde self-assigned this Apr 29, 2026
@ahcorde ahcorde requested a review from arjo129 as a code owner April 29, 2026 09:44
Copy link
Copy Markdown
Contributor

@caguero caguero left a comment

Choose a reason for hiding this comment

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

Should we move the pure non-gazebo functionality to gz-sensors?

@azeey
Copy link
Copy Markdown
Contributor

azeey commented Apr 29, 2026

Should we move the pure non-gazebo functionality to gz-sensors?

This is not an opinion, but I'd encourage us to think about this question in the context of our recent conversations (Gazebo-classic postmortem) about the purpose of gz-sensors and the future goal of integrating it into gz-sim.

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

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

3 participants