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

Hotfix imports for image client #47

Closed
wants to merge 12 commits into from
Closed

Hotfix imports for image client #47

wants to merge 12 commits into from

Conversation

Galoshi
Copy link
Collaborator

@Galoshi Galoshi commented Jun 25, 2024

Fix for #46

cfreundl and others added 12 commits April 24, 2024 14:26
When the amount of data is less than Client.BUF_LEN then try to receive
exactly (number_bytes - total_recved) and not number_bytes, and
Client.BUF_LEN bytes otherwise, so choose the minimum of those two values.

Signed-off-by: Christoph Freundl <[email protected]>
Use the original binary data as much as possible and avoid string
conversions when extracting the required data or counting bytes.

Signed-off-by: Christoph Freundl <[email protected]>
Extend the PCIC format to also receive raw uncompressed images if
available in the application. Depending on the received chunk type the
correct image is constructed for both JPEG data and raw 8-bit pixel data.

Signed-off-by: Christoph Freundl <[email protected]>
This allows to set/get the camera application parameter
"2dUncompressedImages"

Signed-off-by: Christoph Freundl <[email protected]>
…ation.py example for grabbing uncompressed images
@Galoshi Galoshi added bug Something isn't working code improvement Improved code stability labels Jun 25, 2024
@Galoshi Galoshi requested a review from cfreundl June 25, 2024 06:28
@Galoshi Galoshi self-assigned this Jun 25, 2024
@Galoshi Galoshi closed this Jun 25, 2024
@Galoshi Galoshi deleted the hotfix_imports branch June 25, 2024 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working code improvement Improved code stability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants