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

negative bbox coordinate after random transform #988

Closed
sUtomorrow opened this issue May 5, 2019 · 7 comments
Closed

negative bbox coordinate after random transform #988

sUtomorrow opened this issue May 5, 2019 · 7 comments
Labels
stale Issues with no activity for a long time

Comments

@sUtomorrow
Copy link

When the target frame is at the edge of the image, for example, x1, y1, x2, y2 is 0, 0, 200, 200, then after random transform the bbox with keras-retinanet.utils.transform.transform_aabb function, there may cause negative bbox coordinate.

@de-vri-es
Copy link
Contributor

I seem to recall clipping the boxes if that happens. Did this actually cause a crash for you?

@sUtomorrow
Copy link
Author

I seem to recall clipping the boxes if that happens. Did this actually cause a crash for you?

I found the filter_annotations function is call before random_transform_group function, Is there any other function clip the boxes? there is no crash, but it maybe cause anchor select incorrectly, I found this problem when I tried to add anchor-free branch to the model architecture.

@de-vri-es
Copy link
Contributor

I'm not sure. However, there is an old open issue regarding transformations that transform annotations partially out of the image canvas: #223. However, so far nobody has really had any practical issues with the current behaviour so we haven't done anything.

@hgaiser
Copy link
Contributor

hgaiser commented Jun 6, 2019

I seem to recall clipping the boxes if that happens. Did this actually cause a crash for you?

I don't think we clip boxes after random transformations.

@sUtomorrow is this causing an issue for you, or was it just an observation? I see your point on anchor selection, do you think it will cause large enough errors?

@hgaiser
Copy link
Contributor

hgaiser commented Jul 3, 2019

Any update on this @sUtomorrow ?

@sUtomorrow
Copy link
Author

I'm sorry no reply for a long time.
There is no big problem in the code, I just think about if it's well to clip the annotation box(which may be changed by random transform) before select positive/negative anchor box.

@stale
Copy link

stale bot commented Nov 8, 2019

This issue has been automatically marked as stale due to the lack of recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Issues with no activity for a long time label Nov 8, 2019
@stale stale bot closed this as completed Nov 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Issues with no activity for a long time
Projects
None yet
Development

No branches or pull requests

3 participants