1. Don't capture stderr. Nobody needs it, and it seems unlikely that
anybody will.
2. Don't exit the process if the subprocess fails. Since this function
is now (indirectly) used within individual tests, this behavior causes
the testsuite to exit prematurely.
We don't really need to print a custom error message, because the
default exception message already contains the command line and status;
and since we no longer capture stderr, it'll be printed too. Stdout
will, unfortunately, not be printed, but it's not as important for
debugging.
Also, fix an issue where the subprocess status is not checked if
`capture_output` is false.
Add unit tests for define_dependent_job function that is responsible for queuing jobs in chains if they are created for the same user id. Also it removes old integration test that didn't provide enough coverage and was unstable in general.
Co-authored-by: Oleg Valiulin <liam141810@gmail.com>
Co-authored-by: Oleg Valiulin <oleg.valiulin@cvat.ai>
Co-authored-by: Roman Donchenko <roman@cvat.ai>
Typos spell-checker integration
Run `typos` to show typos in project
Run `typos -w` to autofix
Update instructions for running tests:
- for running docker without sudo, the user has to be in docker group
(as per Docker's [Linux
post-install](https://docs.docker.com/engine/install/linux-postinstall/))
- the referenced guide already has instructions for installing the local
packages, no need to duplicate it here
- Now only the visible subset of the points is matched for both
skeletons (previously, the GT skeleton visibility was used)
This conforms with the overall IoU (or accuracy-based) approach to the
quality metrics, used in other comparisons.
- Added protection from invalid numbers in the distance matching
function to avoid similar problems with other annotations
- Updated python test docs about assets updating
Maxim Zhiltsov