Skip to content

Conversation

f-wright
Copy link
Contributor

@f-wright f-wright commented Oct 16, 2025

This PR adds a new bounding boxes label enum to the SDK. It will be used to correctly construct responses on the edge, like Label is used for binary mode. I also bumped the version number to do a new release with this enum.

Due to compatibility issues between datamodel-code-generator and the version of pydantic in use, I also update datamodel-code-generator.

@f-wright f-wright marked this pull request as ready for review October 20, 2025 16:29
@f-wright f-wright requested a review from brandon-wada October 20, 2025 16:29
Copy link
Collaborator

@brandon-wada brandon-wada left a comment

Choose a reason for hiding this comment

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

The file you really want updated is generated/model.py, but it looks like that didn't get updated somehow?

Copy link
Collaborator

@brandon-wada brandon-wada left a comment

Choose a reason for hiding this comment

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

I love the sheer delta in the number of lines edited, looks good.

Only thing that would cause me pause is the introduction of AwareDatetime, but we have a test on AccountMonthToDateInfo, lets just make sure that passes and we should be good

@f-wright f-wright merged commit 2c6d371 into main Oct 20, 2025
8 checks passed
@f-wright f-wright deleted the bbox-enum branch October 20, 2025 23:33
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.

2 participants