HumanSignal / label-studio-converter

Tools for converting Label Studio annotations into common dataset formats
https://labelstud.io/
255 stars 132 forks source link

feat: LSDV-4830: Add option to provide image dimensions in YOLO import so we don't have to open image #202

Closed Axel-Jacobsen closed 1 year ago

Axel-Jacobsen commented 1 year ago

When converting a large number of labels into a tasks.json, especially when the data is mounted via a remote drive, the image opening process can be painfully slow. There are times where

I made these changes to a custom branch of label-studio-converter, but I figured that y'all may like the changes too.

Also, when will the next release be? The YOGO image path fix would be great for us, we are currently just cloning your repo to get the most up-to-date bugfixes.

Note: your contributing docs suggested formatting w/ black --skip-string-normalization, so that is why there are many more diffs than one would expect for this.

codecov-commenter commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b28b062). Click here to learn what that means. Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #202 +/- ## ========================================= Coverage ? 44.17% ========================================= Files ? 20 Lines ? 1707 Branches ? 0 ========================================= Hits ? 754 Misses ? 953 Partials ? 0 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=heartexlabs). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=heartexlabs)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

makseq commented 1 year ago

@Axel-Jacobsen Do we have issue dedicated to this PR? If yes, please mention it here.