label-studio icon indicating copy to clipboard operation
label-studio copied to clipboard

fix: BROS-23: Don't fail on parsing errors in storage

Open niklub opened this issue 6 months ago • 5 comments

niklub avatar May 19 '25 13:05 niklub

Deploy Preview for label-studio-docs-new-theme ready!

Name Link
Latest commit 4ac615957143f5487fa766f50f0d47cb36821c62
Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/682b38094854d400089e3561
Deploy Preview https://deploy-preview-7567--label-studio-docs-new-theme.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify[bot] avatar May 19 '25 13:05 netlify[bot]

Deploy Preview for label-studio-storybook ready!

Name Link
Latest commit 4ac615957143f5487fa766f50f0d47cb36821c62
Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/682b3809f9eb0900083f0d2c
Deploy Preview https://deploy-preview-7567--label-studio-storybook.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify[bot] avatar May 19 '25 13:05 netlify[bot]

Deploy Preview for heartex-docs ready!

Name Link
Latest commit 4ac615957143f5487fa766f50f0d47cb36821c62
Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/682b38099f6e3700085bb240
Deploy Preview https://deploy-preview-7567--heartex-docs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify[bot] avatar May 19 '25 13:05 netlify[bot]

🔍 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
_scan_and_create_links ValueError: Error loading JSON from file "IMAGES/LOWCOSTAI/detection/test/images/00000.png". io...
Event Count: 1.0k

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

sentry[bot] avatar May 19 '25 13:05 sentry[bot]

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 77.81%. Comparing base (132661d) to head (4ac6159). Report is 321 commits behind head on develop.

:warning: Parser warning

The parser emitted a warning. Please review your JUnit XML file:
Warning while parsing testcase attributes: Limit of string is 1000 chars, for name, we got 1222 at 1:45997 in /home/runner/work/label-studio/label-studio/label_studio/functional.xml

For more help, visit our troubleshooting guide.

Files with missing lines Patch % Lines
label_studio/io_storages/base_models.py 60.00% 2 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #7567      +/-   ##
===========================================
- Coverage    77.81%   77.81%   -0.01%     
===========================================
  Files          197      197              
  Lines        15887    15888       +1     
===========================================
  Hits         12363    12363              
- Misses        3524     3525       +1     
Flag Coverage Δ
pytests 77.81% <60.00%> (-0.01%) :arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

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

codecov[bot] avatar May 19 '25 14:05 codecov[bot]

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

robot-ci-heartex avatar Jul 04 '25 02:07 robot-ci-heartex

This PR was closed because it has been stalled for 10 days with no activity.

robot-ci-heartex avatar Jul 14 '25 02:07 robot-ci-heartex