HumanSignal / label-studio

Label Studio is a multi-type data labeling and annotation tool with standardized output format
https://labelstud.io
Apache License 2.0
19.44k stars 2.41k forks source link

fix: OPTIC-1373: Update async import and reimport to be on by default and amend locust tests #6696

Closed bmartel closed 21 hours ago

bmartel commented 1 day ago

PR fulfills these requirements

Change has impacts in these area(s)

(check all that apply)

Describe the reason for change

Locust tests failed to run with async import and reimport on, staling the FFs involved and updating the tests to be compatible.

What feature flags were used in this change?

Does this PR introduce a breaking change?

(check only one)

What level of testing was included in the change?

(check all that apply)

Which logical domain(s) does this change affect?

Async Import/Reimport

netlify[bot] commented 1 day ago

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

Name Link
Latest commit fe490a884a73860e17cb252ebeaeac5f73860132
Latest deploy log https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/673f517bc231ce00084d1329
netlify[bot] commented 1 day ago

Deploy Preview for heartex-docs canceled.

Name Link
Latest commit fe490a884a73860e17cb252ebeaeac5f73860132
Latest deploy log https://app.netlify.com/sites/heartex-docs/deploys/673f517be3d02600089e3fbd
codecov[bot] commented 1 day ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 76.77%. Comparing base (86c907c) to head (fe490a8). Report is 5 commits behind head on develop.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #6696 +/- ## ======================================== Coverage 76.77% 76.77% ======================================== Files 170 170 Lines 13883 13883 ======================================== Hits 10659 10659 Misses 3224 3224 ``` | [Flag](https://app.codecov.io/gh/HumanSignal/label-studio/pull/6696/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HumanSignal) | Coverage Ξ” | | |---|---|---| | [pytests](https://app.codecov.io/gh/HumanSignal/label-studio/pull/6696/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HumanSignal) | `76.77% <ΓΈ> (ΓΈ)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HumanSignal#carryforward-flags-in-the-pull-request-comment) to find out more.

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


🚨 Try these New Features: