Skip to content

Conversation

@marco-paladini
Copy link

Moving the luxonis/depthai-python#655 pull request to the experiments repo.

This PR provides users with a debugging program that computes a point cloud and writes all debugging data to disk.
Also added a minimal pcd viewer and a README for the demo program.

@Luxonis-Brandon
Copy link
Contributor

Thank you!

@marco-paladini
Copy link
Author

This demo program was based off rgb_depth_aligned but now I noticed the rgbd-pointcloud program in this repo does mostly everything that is done in save_point_cloud except for saving the data to disk, so potentially this PR should be closed and the save code added to rgbd-pointcloud?

frameDepth = None

if cv2.waitKey(1) == ord("q"):
# save all images
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe we could save images when pressing s as save instead?

Comment on lines +146 to +153
queueEvents = device.getQueueEvents(
("rgb", "depth", "rectifiedLeft", "rectifiedRight")
)
for queueName in queueEvents:
packets = device.getOutputQueue(queueName).tryGetAll()
if len(packets) > 0:
latestPacket[queueName] = packets[-1]

Copy link
Contributor

Choose a reason for hiding this comment

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

This logic of getting frames isn't ideal. Maybe jut use get() instead, or maybe add syncing logic to it (based on sequence number)?

stereo.rectifiedRight.link(xoutRectifRight.input)

# Connect to device and start pipeline
with device:
Copy link
Contributor

Choose a reason for hiding this comment

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

since you are using context manager, move with dai.Device(pipeline) as device here (like in all other demos)

@Erol444
Copy link
Contributor

Erol444 commented Sep 13, 2022

@marco-paladini yes I agree I would just expand the existing demo to add saving option when pressing eg. s key. Thanks!

@klemen1999
Copy link
Collaborator

Added in #437

@klemen1999 klemen1999 closed this Jul 21, 2025
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

Successfully merging this pull request may close these issues.

4 participants