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

Bounding Box tool for rectangular segmentation. #226

Merged
merged 18 commits into from
Aug 19, 2019

Conversation

mubashirhanif
Copy link
Contributor

@mubashirhanif mubashirhanif commented Jul 15, 2019

This tool allows to segment the images in the bounding box fashion, and some extra functionality, as mentioned in closes #164 .

There are a couple of assumptions:

  1. Only one bounding box per annotation instance.
  2. The bounding box co-ordinates(4 corners) can be found under segmentation attribute of the annotation in one of permutation of the following array. [UpperLeft, UpperRight, BottomRight, BottomLeft], though opposing corners are always stored 2 indices apart. Such as if UpperLeft is at index 0 then BottomRight will be on index 2.

This feature also adds some additional functionality to SelectTool such as:

  1. You can now move the annotated bounding box, In future I'll try to add this functionality for polygon as well.
  2. You can resize the bounding box in bounding box fashion.

@jsbroks jsbroks added the status: in progress Issue is being worked on label Jul 23, 2019
@jsbroks jsbroks merged commit 16ab496 into jsbroks:master Aug 19, 2019
@GeorgeSeif
Copy link

Is there any tutorial on how to use this tool?

@rbavery
Copy link
Contributor

rbavery commented Sep 2, 2019

I'm also curious how to enable this, I just started using the most recent version today, but can't figure out how to use the bounding box tool and if moving polygons with select is implemented. awesome tool and thanks so much for developing it!

@jsbroks
Copy link
Owner

jsbroks commented Sep 2, 2019

Moving a polygon is not implemented. You can move points of a polygon but not the entire ploygon.

Are you using the stable docker image or lastest

@mubashirhanif
Copy link
Contributor Author

@rbavery the polygon movement is not yet implemented, I'll work on that feature as well once I have time. One problem in implementing the moving for polygon, is the way the polygon and HitTest works, for e.g. since the polygons have multiple closed components in them I don't know which closed component was selected so that I can move it. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: in progress Issue is being worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Addition of additional tools - RectangularBox!
5 participants