Skip to content

Conversation

@dlangenk
Copy link
Member

Allow to filter largo patches by the size of the annotation in px² as proposed in #1166.

@mzur
Copy link
Member

mzur commented Jul 23, 2025

Please not more than one AI generated PR at a time 😄

Copy link
Member

@mzur mzur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So I finally had time to think about this more. Ignoring the fact that the complicated SQL queries are duplicated all over the place, I'm sure that they will overwhelm the production database. We should test this if we move forward with this approach.

But here is an alternative idea. What about adding an indexed size attribute to the image/video_annotations tables? The size is computed once when the annotation is created (and when it is updated) and Largo can use this to filter the annotations much more efficiently.

This will have a substantial impact on the size of the tables so we have to be sure that this feature is important enough for this change. Or maybe there are other feature ideas that could benefit from a new size attribute?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants