-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Document clarification for different representation for bounding box between Turi Create and Vision #2320
Conversation
@@ -99,7 +99,8 @@ def _find_only_column_of_type(sframe, target_type, type_name, col_name): | |||
raise ToolkitError('No "{col_name}" column specified and more than one {type_name} column in "dataset". Can not infer correct {col_name} column.'.format(col_name=col_name, type_name=type_name)) | |||
image_column_name = name | |||
if image_column_name is None: | |||
raise ToolkitError('No %s column in "dataset".' % type_name) | |||
raise ToolkitError('No "{col_name}" column specified and no column with expected type "{type_name}" is found.'.format(col_name=col_name, type_name=type_name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like a fix for an unrelated bug (unrelated to "Documentation clarification for OD")
In general, you should avoid mixing together branches for different bugs. Each PR should just merge the changes/commits for the issue it's fixing!
@@ -37,6 +37,8 @@ For more details on the integration with Core ML and a sample app to get | |||
you started, please look at the the article on | |||
[Recognizing Objects in Live Capture](https://developer.apple.com/documentation/vision/recognizing_objects_in_live_capture). | |||
|
|||
**Note:** The bounding box object `VNRecognizedObjectObservation.boundingBox` has a different definition from the one used for Turi Create. First, the location of the bounding box is defined by the upper-left corner of the rectangle instead of the center. Secondly, the location, width, and height for the bounding box are all normalized between [0,1] by the dimensions of the images. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe Vision uses the lower left corner as the origin of the coordinates, and the lower-left corner of the bounding box to define the position. Is this not what you're observing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just changed the documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Close #1016
close #1984