Skip to content
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

[YOLO]Add the probability to the image #1910

Merged
merged 2 commits into from
Oct 18, 2018
Merged

Conversation

siju-samuel
Copy link
Member

@siju-samuel siju-samuel commented Oct 17, 2018

Add the probability along with the item name in the image, after drawing the bounding box

image

@PariksheetPinjari909 please review. Thanks

Thanks for contributing to TVM! Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from others in the community.

Add the probability along with the item name in the image, after drawing the bounding box
@@ -160,7 +160,7 @@ def draw_detections(im, dets, thresh, names, classes):
if det['prob'][j] > thresh:
if category == -1:
category = j
labelstr.append(names[j])
labelstr.append(names[j] + "(" + str(round(det['prob'][j]*100, 2)) + ")")
Copy link
Contributor

@PariksheetPinjari909 PariksheetPinjari909 Oct 18, 2018

Choose a reason for hiding this comment

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

Yes, it will better for user to know the probability in the image itself. Either you add a % sign or keep probability as it is upto 4 decimals

@tqchen tqchen merged commit f1a4b94 into apache:master Oct 18, 2018
FrozenGene pushed a commit to FrozenGene/tvm that referenced this pull request Dec 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants