-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
fix: LEAP-1004: Show prediction with selected project.model_version in QuickView #5729
Conversation
✅ Deploy Preview for heartex-docs canceled.
|
✅ Deploy Preview for label-studio-docs-new-theme canceled.
|
Why do we think this is a correct behavior? That should be an EXPLICIT user action to select what predictions should be used? |
/git merge develop
|
In the current implementation, the only explicit user action is to set the model version in project settings, which produces the bug fixed in this PR. Two alternatives that could be better from UX perspective:
Is there any other scenario that can better address the problem? Anyway, this PR obviously fixes an incorrect behavior, and we can address future enhancements in the next ones |
In "QuickView" mode, user now sees the prelabeled data from the prediction based on selected model version.
If prediction from this model is not presented it fallbacks to old behavior and uses the first prediction.
Model version selected:
Correct prediction is shown first:
PR fulfills these requirements
Change has impacts in these area(s)