-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Patch RichLabelAnnotator, fix in-place functionality of other annotators #1277
Conversation
@LinasKo I see a lot of |
No. I'll do that, will let you know the results. If we prefer to optimize, I can do that. |
The Colab now has tests in the end, comparing the performance on a 10,000x10,000 size empty image. With the latest version, performance is equivalent or slightly better than in base supervision. |
Hey @onuralpszr , Given you've worked on it before, would you mind reviewing these changes too? There's a colab that shows how it works, only requiring that you upload the font file. |
@LinasKo start reviewing |
Description
This is a 3-in-1:
RichLabelAnnotator
class #1116np.ndarray
after annotationType of change
Please delete options that are not relevant.
How has this change been tested, please provide a testcase or example of how you tested the change?
Ran every annotator except for TraceAnnotator.
Ran tests.
Colab: https://colab.research.google.com/drive/1NvVs28MnSb-_W-zkFa6-FXiY-0G4nx7k#scrollTo=dVZIKNV1uXVl
Any specific deployment considerations
Docs