-
Notifications
You must be signed in to change notification settings - Fork 180
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
DeepForest support for multiple annotation types (point, box, polygon). #590
Conversation
@henrykironde and @ethanwhite I invite thoughts on how we proceed. My plan is to work on this branch while developing MillionTrees dataset https://milliontrees.idtrees.org/ and when I feel like the kinks are worked out, trigger the PR. This would be enough for DeepForest 1.4.0. |
e823400
to
1a15f77
Compare
Add better release notes. |
Add support for multiple annotation types (point, box, polygon)
This looks good in terms of logic. There are a few changes needed, but I will handle them later as part of a general cleaning and update. |
We want to prepare DeepForest to be able to ingest annotation geometries covering point, box and polygon formats. This is the first step to allow models and predictions in each and among each format. I think this is crucial for accessing every possible dataset available. This was spurred during development of the MillionTrees benchmark. Here is a roadmap
Pros:
Cons:
https://github.com/weecology/DeepForest/blob/b87f9314e666903bb361c614e24daf53e5abad03/deepforest/predict.py#L112C16-L112C16
DeepForest/deepforest/predict.py
Line 69 in b87f931
and many more. We will need to first check what geometry type we have and perform the correct transformations.
Roadmap for merge
Long term goals