Skip to content

Conversation

@JBWilkie
Copy link
Contributor

Problem

A previous PR introduced in 0.8.45 broke the following annotation types when converting from 2.0 to 1.0 JSON:

  • Cuboid
  • Ellipse
  • Keypoint
  • Skeleton

Solution

  • Introduced logic to handle these types during conversion
  • Added a test for this logic

Changelog

Fixed a bug that prevented certain annotation types from being converted from 2.0 to 1.0 JSON

@linear
Copy link

linear bot commented Jan 23, 2024


if not v1_data:
v1_data[annotation_class.annotation_type] = data

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is how the entire function used to work. Basically if it's not a tag, bbox, or polygon, simply return the structure that we used to

Copy link
Member

@saurbhc saurbhc left a comment

Choose a reason for hiding this comment

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

LGTM!

@JBWilkie JBWilkie merged commit df1219e into master Jan 30, 2024
@saurbhc saurbhc deleted the DAR-214 branch January 30, 2024 10:20
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.

3 participants