-
Notifications
You must be signed in to change notification settings - Fork 199
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
first #67
base: main
Are you sure you want to change the base?
first #67
Conversation
|
||
# Plot the annotated image from the Result object | ||
# Include the confidence value | ||
image_annotated = ... | ||
image_annotated = prediction.plot() |
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.
Please include the confidence value in the image (use the conf kwarg)
|
||
for i in range(boxes_cpu.shape[0]): | ||
results, bbox = bounding_box.BoundingBox.create(boxes_cpu[i]) | ||
if not results: |
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.
Normally in statistics, we would discard the entire data point (the whole image and all the bounding boxes) in case one part of it (one bounding box) fails. (You do not need to change anything, just something to think about)
self.waypoint.location_y - report.position.location_y | ||
) | ||
self.traveled_to_waypoint = True | ||
elif not self.closest_landing_pad: |
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.
Using this as a statement to check if you have found the closest landing pad or not can be a little dangerous, in the sense that if there were no landing pads in the list (ie it was empty), you wouldn't know that you had already checked that it was empty. (You do not need to change anything, just something to think about)
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.
Reviewed
No description provided.