Skip to content

Comments

feat: DIA-2148: write exports using thread pool#7342

Merged
pakelley merged 7 commits intodevelopfrom
fb-dia-2148
Apr 18, 2025
Merged

feat: DIA-2148: write exports using thread pool#7342
pakelley merged 7 commits intodevelopfrom
fb-dia-2148

Conversation

@pakelley
Copy link
Contributor

@pakelley pakelley commented Apr 8, 2025


Optimize export job by threading writes to storage

@github-actions github-actions bot added the feat label Apr 8, 2025
@netlify
Copy link

netlify bot commented Apr 8, 2025

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

Name Link
🔨 Latest commit 70ff4cc
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/68003eed681aaf0008d98183

@netlify
Copy link

netlify bot commented Apr 8, 2025

Deploy Preview for label-studio-storybook canceled.

Name Link
🔨 Latest commit 70ff4cc
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-storybook/deploys/68003eedfa2756000861f97b

@netlify
Copy link

netlify bot commented Apr 8, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 70ff4cc
🔍 Latest deploy log https://app.netlify.com/sites/heartex-docs/deploys/68003eed68703900085b1c67

@sentry
Copy link

sentry bot commented Apr 8, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: label_studio/io_storages/base_models.py

Function Unhandled Issue
save_all_annotations S3StorageError: Debugging info is not available for s3 endpoints on domain: . Please contact your Label Studio devops team if you require detailed error reporting for this domain. ...
Event Count: 39

Did you find this useful? React with a 👍 or 👎

@codecov
Copy link

codecov bot commented Apr 8, 2025

Codecov Report

Attention: Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 77.26%. Comparing base (8714cb3) to head (70ff4cc).
Report is 16 commits behind head on develop.

Files with missing lines Patch % Lines
label_studio/io_storages/base_models.py 95.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #7342      +/-   ##
===========================================
- Coverage    77.27%   77.26%   -0.01%     
===========================================
  Files          190      190              
  Lines        14682    14697      +15     
===========================================
+ Hits         11345    11356      +11     
- Misses        3337     3341       +4     
Flag Coverage Δ
pytests 77.26% <95.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:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pakelley
Copy link
Contributor Author

pakelley commented Apr 16, 2025

/fm sync

Workflow run

@pakelley pakelley merged commit 78198c7 into develop Apr 18, 2025
49 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-dia-2148 branch April 18, 2025 01:17
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