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)

@sktometometo sktometometo force-pushed the PR/spot_arm/support-additional-hand-image-compression branch from f75fe9f to 3a03187 Compare November 22, 2023 03:07
@github-actions github-actions bot added the spot label Nov 22, 2023
@k-okada k-okada force-pushed the spot_arm branch 4 times, most recently from 4d64e9f to 7908647 Compare November 20, 2024 01:38
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