Skip to content

Conversation

bmartel
Copy link
Contributor

@bmartel bmartel commented Oct 3, 2025

This pull request updates the task selection logic in label_studio/projects/functions/next_task.py to ensure that ground truth (GT) annotations are not prioritized when using the breadth-first strategy, unless the project is in onboarding mode. The main changes involve updating the _try_breadth_first function to accept the project parameter and modifying the annotation counting logic to exclude GT annotations when appropriate.

Task selection logic improvements:

  • Updated _try_breadth_first to accept a project parameter and adjusted the annotation counting to exclude ground truth annotations when the project is not set to show ground truth tasks first, preventing GT tasks from being prioritized in the breadth-first queue.
  • Updated calls to _try_breadth_first in get_next_task_without_dm_queue to pass the project parameter.

Copy link

netlify bot commented Oct 3, 2025

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit 5adf2c5
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/68e3f9704303040008c33dd1

Copy link

netlify bot commented Oct 3, 2025

Deploy Preview for label-studio-storybook canceled.

Name Link
🔨 Latest commit 5adf2c5
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/68e3f970df5dcd0008f6865a

@github-actions github-actions bot added the fix label Oct 3, 2025
Copy link

netlify bot commented Oct 3, 2025

Deploy Preview for label-studio-playground canceled.

Name Link
🔨 Latest commit 5adf2c5
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/68e3f970b553000008f80c58

Copy link

netlify bot commented Oct 3, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit c746770
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/68e03e65b74e9b0008437226

Copy link

netlify bot commented Oct 3, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 5adf2c5
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/68e3f9702518cf0008fb4850

Copy link

codecov bot commented Oct 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.69%. Comparing base (db4880f) to head (5adf2c5).
⚠️ Report is 3 commits behind head on develop.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop    #8587       +/-   ##
============================================
+ Coverage    67.41%   79.69%   +12.28%     
============================================
  Files          782      238      -544     
  Lines        60005    21540    -38465     
  Branches     10161        0    -10161     
============================================
- Hits         40451    17167    -23284     
+ Misses       19551     4373    -15178     
+ Partials         3        0        -3     
Flag Coverage Δ
lsf-e2e ?
lsf-integration ?
lsf-unit ?
pytests 79.69% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bmartel bmartel requested a review from makseq October 6, 2025 17:07
@bmartel
Copy link
Contributor Author

bmartel commented Oct 6, 2025

/fm sync

Workflow run

@robot-ci-heartex robot-ci-heartex merged commit 179fbdb into develop Oct 7, 2025
53 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-fit-756 branch October 7, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants