-
Notifications
You must be signed in to change notification settings - Fork 3.1k
fix: ROOT-223: Optimize "Delete predictions" DM action #8700
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
Conversation
✅ Deploy Preview for heartex-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for label-studio-storybook ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for label-studio-playground ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for label-studio-docs-new-theme ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #8700 +/- ##
============================================
+ Coverage 67.04% 79.78% +12.74%
============================================
Files 790 239 -551
Lines 60686 21691 -38995
Branches 10316 0 -10316
============================================
- Hits 40684 17306 -23378
+ Misses 19999 4385 -15614
+ Partials 3 0 -3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/fm sync |
|
/git merge develop
|
|
/git merge develop
|
|
/git merge develop
|
| project = generics.get_object_or_404(Project, pk=pk) | ||
| self.check_object_permissions(request, project) | ||
|
|
||
| queryset = get_prepared_queryset(request, project) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not keep this codepath unchanged if the FF is off?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ff-off path is identical to the body of get_prepared_queryset, I figured the effect of the change is more readable this way
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
label-studio/label_studio/data_manager/functions.py
Lines 329 to 332 in b52eb9d
| def get_prepared_queryset(request, project): | |
| prepare_params = get_prepare_params(request, project) | |
| queryset = Task.prepared.only_filtered(prepare_params=prepare_params) | |
| return queryset |
It's only used in one other place, could be worth removing this function entirely if we end up touching the other callsite
|
/git merge develop
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, cool idea of cutting ordering from delete queries 👍
|
/git merge develop
|
Shuffle around DB queries for this codepath