Barometer units - Githubissues

stereolabs / zed-ros-wrapper

ROS wrapper for the ZED SDK

https://www.stereolabs.com/docs/ros/

MIT License

447 stars 391 forks source link

Barometer units #854

Closed arenas7307979 closed 1 year ago

arenas7307979 commented 2 years ago

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, bug report, or anything other than a feature request directly related to this project.

Proposal

hii, about zed-ros-wrapper what is units for topic of /zed2i/zed_node/atm_press

testing on the second floor below ground, and the result doesn't look right

Use-Case

No response

Anything else?

No response

yhabib29 commented 2 years ago

It seems to be converted in Pascal here zed_wrapper_nodelet.cpp:2964

arenas7307979 commented 2 years ago

The chart is my downstairs data. It looks like the unit should be Pascal*100, but the press change of ZED2i is very bad. The chart on the right side is the barometer of my phone

Myzhar commented 1 year ago

Hi @arenas7307979 the ROS units for the pressure is Pa, according to the topic definition. I agree that there is a bug in units conversion, we're fixing it.

Regarding the measurement. The sensor is inside a closed case, so you must take it into consideration.