Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix(python): default ImageResponse buffers to empty bytes/list#9810

Open
Bartok9 wants to merge 1 commit into
microsoft:mainmicrosoft/AirSim:mainfrom
Bartok9:aerial/2026-07-24-image-response-defaultsBartok9/AirSim:aerial/2026-07-24-image-response-defaultsCopy head branch name to clipboard
Open

fix(python): default ImageResponse buffers to empty bytes/list#9810
Bartok9 wants to merge 1 commit into
microsoft:mainmicrosoft/AirSim:mainfrom
Bartok9:aerial/2026-07-24-image-response-defaultsBartok9/AirSim:aerial/2026-07-24-image-response-defaultsCopy head branch name to clipboard

Conversation

@Bartok9

@Bartok9 Bartok9 commented Jul 24, 2026

Copy link
Copy Markdown

Summary

Default ImageResponse.image_data_uint8 / image_data_float were scalar placeholders (np.uint8(0) and 0.0). Sample code and helpers call len(...) and np.frombuffer on those fields; empty bytes and an empty list match the RPC vector payloads.

Motivation

Prevents TypeError on cold construction and documents the intended container shapes for clients that inspect responses before a capture fills them.

Verification

  • Offline unittest PythonClient/tests/test_image_response_types.py (package import needs msgpackrpc when deps present).
  • Static review vs C++ RpcLibAdaptorsBase::ImageResponse vectors.

AI-assisted scoping; human-reviewed micro-diff.

Claim: bartok
Operator: bartok
Campaign: aerial-drone

image_data_uint8 was np.uint8(0) and image_data_float was 0.0, which breaks
len()/frombuffer paths used throughout PythonClient samples. Align with
vector-shaped RPC payloads.
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.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.