Skip to content

feat: ROOT-9: Allow reading multiple tasks from a JSON file in source cloud storage #7540

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.

Sign up for GitHub

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

Jump to bottom
Merged
matt-bernstein merged 27 commits into from
May 16, 2025
Merged

feat: ROOT-9: Allow reading multiple tasks from a JSON file in source cloud storage #7540

matt-bernstein merged 27 commits into from
May 16, 2025

Conversation

@matt-bernstein
Copy link
Contributor

@matt-bernstein matt-bernstein commented May 14, 2025

Fix all places that assume key is unique in StorageLink

matt-bernstein and others added 13 commits May 14, 2025 12:15
@matt-bernstein
feat: ROOT-1: Add offset to storage link model to support storage lin…
3d23bf8 
…k to task within file of tasks on cloud storage
@matt-bernstein
review comments
9e2a7bb
@matt-bernstein @jombooth
remove whitespace
2cbcb41 
Co-authored-by: Jo Booth <[email protected]>
@matt-bernstein @jombooth
update comment
3de9ffb 
Co-authored-by: Jo Booth <[email protected]>
@matt-bernstein
Sync Follow Merge dependencies
dc423d0 
Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/15027678115
@matt-bernstein
Merge branch 'develop' into 'fb-ROOT-1'
ad6a221 
Workflow run: https://github.com/HumanSignal/label-studio/actions/runs/15027678115
@matt-bernstein
feat: ROOT-9: Allow reading multiple tasks from a JSON file in source…
411940c 
… cloud storage
@matt-bernstein
update comment
f37b8af
@matt-bernstein
update storagelink creation
cfbb076
@matt-bernstein
fix DM action
d3c02d9
@matt-bernstein
blue
1033d26
@matt-bernstein
Merge remote-tracking branch 'origin/develop' into fb-ROOT-9
c18d792
@matt-bernstein
udpate test
4e75268
@matt-bernstein matt-bernstein requested a review from a team as a code owner May 14, 2025 19:44
@netlify
Copy link

netlify bot commented May 14, 2025 •
edited
Loading

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

Name Link
🔨 Latest commit 29d8944
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/6827a7b540f85c0008a0792c

@netlify
Copy link

netlify bot commented May 14, 2025 •
edited
Loading

✅ Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 29d8944
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/6827a7b57584900008dc71dd

@netlify
Copy link

netlify bot commented May 14, 2025 •
edited
Loading

✅ Deploy Preview for label-studio-storybook canceled.

Name Link
🔨 Latest commit 29d8944
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/6827a7b59dde3b0008ddf6dd

@sentry
Copy link

sentry bot commented May 14, 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
_scan_and_create_links ValueError: Error loading JSON from file "IMAGES/LOWCOSTAI/detection/test/images/00000.png". io...
Event Count: 18

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

@github-actions github-actions bot added the feat label May 14, 2025
@codecov
Copy link

codecov bot commented May 14, 2025 •
edited
Loading

⚠️ 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

@matt-bernstein matt-bernstein requested a review from jombooth May 14, 2025 20:44
@matt-bernstein
Copy link
Contributor Author

matt-bernstein commented May 14, 2025 •
edited by robot-ci-heartex
Loading

/fm sync

Workflow run

@matt-bernstein
Copy link
Contributor Author

matt-bernstein commented May 15, 2025 •
edited by robot-ci-heartex
Loading

/git merge develop

Workflow run
Successfully merged: 2 files changed, 9 insertions(+), 7 deletions(-)

@matt-bernstein
Copy link
Contributor Author

matt-bernstein commented May 15, 2025 •
edited by robot-ci-heartex
Loading

/fm sync

Workflow run

@matt-bernstein
Copy link
Contributor Author

matt-bernstein commented May 15, 2025 •
edited by robot-ci-heartex
Loading

/fm sync

Workflow run

jombooth
jombooth reviewed May 15, 2025
View reviewed changes
@matt-bernstein @jombooth
typo
f195422 
Co-authored-by: Jo Booth <[email protected]>
@matt-bernstein matt-bernstein mentioned this pull request May 16, 2025
Merged
@jombooth
Copy link
Contributor

jombooth commented May 16, 2025 •
edited by robot-ci-heartex
Loading

/git merge develop

Workflow run
Successfully merged: create mode 100644 web/libs/ui/src/lib/auto-sizer-table/auto-sizer-table.tsx

@matt-bernstein matt-bernstein enabled auto-merge (squash) May 16, 2025 21:05
@matt-bernstein matt-bernstein merged commit 6da1356 into develop May 16, 2025
45 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-ROOT-9 branch May 16, 2025 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@jombooth jombooth jombooth approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

feat FM Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@matt-bernstein @jombooth @robot-ci-heartex