-
Notifications
You must be signed in to change notification settings - Fork 70
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
Non Normalized coordinates #30
Comments
Yeah, |
@vaishnavm217 to fix this do we just do something like this? to normalise the bounding boxes. w,h = image.size
boxes /= torch.Tensor([w,h,w,h]).expand_as(boxes) |
That's more appropriate.
But assuming a square image as input, you can just divide it by any one of
the dimension.
…On Sun, 15 Jul 2018, 10:22 Leon, ***@***.***> wrote:
@vaishnavm217 <https://github.com/vaishnavm217> to fix this do we just do
something like this? to normalise the bounding boxes.
w,h = image.size
boxes /= torch.Tensor([w,h,w,h]).expand_as(boxes)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#30 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AQy9UHw_G5TtJTRyZOSs4DGpGYW8B_O4ks5uGsqmgaJpZM4UlnKK>
.
|
@vaishnavm217 thank you for the confirmation. much appreciated. I'm testing out the train now and it's working well. Just looking ahead. When I run it through prediction or evaluation, where would the best place to denormalise the labels?. |
Hi,
I am working on face detection using your ssd model. One thing I noticed that my model was giving zero loss, i.e all the ious were zero. After long inspection
torchcv/examples/ssd/train.py
Line 63 in 6291f3e
this line passes the boxes coordinates as it is. It should be normalized to get correct iou
The text was updated successfully, but these errors were encountered: