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

v2-main branch does not initialize filters. #83

Open
john-maidbot opened this issue Dec 17, 2024 · 3 comments
Open

v2-main branch does not initialize filters. #83

john-maidbot opened this issue Dec 17, 2024 · 3 comments

Comments

@john-maidbot
Copy link

Hello,

At first I noticed that decimation filter did not seem to be working. When I looked closer, no filters work atm on v2-main.
It appears there is a typo (

if (!filter_list_.empty()) {
) that prevents the filters from being setup. It is returning early if the list is not empty. but it should be the reverse.

OS:
Ubuntu 22.04

Model:
Gemini 335

Software:
OrbbecSDK_ROS2, v2-main branch

Node parameters checked at runtime:
decimation_filter_scale_: 2
enable_decimation_filter: true
depth_camera_info_qos: default
depth_format: ANY
depth_fps: 10
depth_height: 480
depth_registration: false
depth_width: 640
enable_depth: true
enable_depth_scale: true

Expectation:

The published depth image should be 320 x 240 (i.e. 640 / 2 x 480 / 2)

Actual behavior:

ros2 topic echo /camera/depth/image_raw
header:
  stamp:
    sec: 1734464379
    nanosec: 723162880
  frame_id: camera_depth_optical_frame
height: 480
width: 640
encoding: 16UC1
is_bigendian: 0
step: 1280
data:
...

I can make a PR if that would be helpful?

Thank you for your time.

@jjiszjj
Copy link
Collaborator

jjiszjj commented Dec 19, 2024

@john-maidbot Thanks for the feedback, I will do a check on v2-main

@jjiszjj
Copy link
Collaborator

jjiszjj commented Dec 19, 2024

@john-maidbot After our debugging, we found that the problem you described does exist. The modification you described is correct. Thank you for your feedback during use. We will fix this error in the next version update.

@john-maidbot
Copy link
Author

thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants