openvinotoolkit / datumaro

Dataset Management Framework, a Python library and a CLI tool to build, analyze and manage Computer Vision datasets.
https://openvinotoolkit.github.io/datumaro/
MIT License
551 stars 135 forks source link

Handle path separators in the subset when exporting a datumaro dataset #1615

Closed jihyeonyi closed 2 months ago

jihyeonyi commented 2 months ago

Summary

Ticket: 152928 Raise an error when exporting a dataset in Datumaro format if the subset name contains path separators.

How to test

Checklist

License

# Copyright (C) 2024 Intel Corporation
#
# SPDX-License-Identifier: MIT
codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 95.45455% with 1 line in your changes missing coverage. Please review.

Project coverage is 81.07%. Comparing base (ff5fd94) to head (59074ac). Report is 4 commits behind head on develop.

Files with missing lines Patch % Lines
src/datumaro/components/errors.py 80.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #1615 +/- ## =========================================== + Coverage 81.06% 81.07% +0.01% =========================================== Files 278 278 Lines 32517 32588 +71 Branches 6607 6620 +13 =========================================== + Hits 26360 26422 +62 - Misses 4701 4710 +9 Partials 1456 1456 ``` | [Flag](https://app.codecov.io/gh/openvinotoolkit/datumaro/pull/1615/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openvinotoolkit) | Coverage Δ | | |---|---|---| | [ubuntu-20.04_Python-3.10](https://app.codecov.io/gh/openvinotoolkit/datumaro/pull/1615/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openvinotoolkit) | `81.06% <95.45%> (+0.01%)` | :arrow_up: | | [windows-2022_Python-3.10](https://app.codecov.io/gh/openvinotoolkit/datumaro/pull/1615/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openvinotoolkit) | `81.05% <95.45%> (+0.01%)` | :arrow_up: | 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=openvinotoolkit#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.