Skip to content

Conversation

mcvella
Copy link
Collaborator

@mcvella mcvella commented Apr 4, 2025

No description provided.

@hexbabe hexbabe self-requested a review April 4, 2025 18:12
@@ -1,11 +1,11 @@
# Keep the below block the same as requirements.txt
depthai==2.26.0
depthai==2.30.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

why did this need to be updated?

numpy>=1.23,<2.0.0 # >2.0.0 breaks depthai https://github.com/luxonis/depthai-viewer/releases/tag/v0.2.3
google-api-python-client==2.0.0
pyinstaller==6.3.0
pyinstaller==6.12.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

why did this need to be updated?

self.depth_stream_name, MAX_COLOR_DEPTH_QUEUE_SIZE, blocking=False
)
if type(self.cfg) == OakDConfig and self.cfg.point_cloud_enabled:
if isinstance(self.cfg, OakDConfig) and self.cfg.point_cloud_enabled:
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this fixing a bug?

@nicksanford nicksanford merged commit 2f75bfd into viam-modules:main Apr 11, 2025
1 check passed
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.

3 participants