zed-ros-wrapper icon indicating copy to clipboard operation
zed-ros-wrapper copied to clipboard

Barometer units

Open arenas7307979 opened this issue 1 year ago • 1 comments

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 image image

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

Use-Case

No response

Anything else?

No response

arenas7307979 avatar Aug 12 '22 09:08 arenas7307979

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

yhabib29 avatar Aug 15 '22 20:08 yhabib29

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

image

arenas7307979 avatar Aug 17 '22 09:08 arenas7307979

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.

Myzhar avatar Oct 13 '22 08:10 Myzhar