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

Support additional hand image compression #96

Open
wants to merge 2 commits into
base: spot_arm
Choose a base branch
from

Conversation

sktometometo
Copy link

Reflecting k-okada/spot_ros-arm#4 change and publish compressed rgb and depth image for hands.

@sktometometo sktometometo force-pushed the PR/spot_arm/support-additional-hand-image-compression branch from 4302467 to f75fe9f Compare July 4, 2023 07:34
@sktometometo
Copy link
Author

@k-okada Please also merge this

@k-okada
Copy link
Owner

k-okada commented Nov 22, 2023

Plaese check the CPU usage rate. I am concerned that the CPU utilization may become unnecessarily high for a series of partial PRs.
(1) I would originally like to use connectionbasedtransport, but I think the dependency on spot->jsk is too much work, so I would like to put in a similar process and fetch images from inside the body only when necessary.
2) Since it seems that JEPG images can be retrieved using SpotSDK, only compressed images should be published in the first place.
https://dev.bostondynamics.com/python/bosdyn-client/src/bosdyn/client/image#bosdyn.client.image.build_image_request

Translated with www.DeepL.com/Translator (free version)

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

Successfully merging this pull request may close these issues.

2 participants