datumaro icon indicating copy to clipboard operation
datumaro copied to clipboard

Set label name with parents to avoid duplicates for AstypeAnnotations

Open sooahleex opened this issue 9 months ago • 1 comments

Summary

  • Except nan for label
  • Set label name with parents to avoid duplicate name for AstypeAnnotations

How to test

  • Add unit test when tabular dataset included missing value
  • Add unit test for AstypeAnnotations when label value is nan

Checklist

  • [X] I have added unit tests to cover my changes.​
  • [ ] I have added integration tests to cover my changes.​
  • [X] I have added the description of my changes into CHANGELOG.​
  • [ ] I have updated the documentation accordingly

License

  • [ ] I submit my code changes under the same MIT License that covers the project. Feel free to contact the maintainers if that's a concern.
  • [ ] I have updated the license header for each file (see an example below).
# Copyright (C) 2024 Intel Corporation
#
# SPDX-License-Identifier: MIT

sooahleex avatar May 08 '24 07:05 sooahleex

Codecov Report

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

Project coverage is 80.62%. Comparing base (44cc56a) to head (4042077). Report is 58 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1492      +/-   ##
===========================================
- Coverage    80.85%   80.62%   -0.24%     
===========================================
  Files          271      276       +5     
  Lines        30689    31369     +680     
  Branches      6197     6313     +116     
===========================================
+ Hits         24815    25291     +476     
- Misses        4489     4663     +174     
- Partials      1385     1415      +30     
Flag Coverage Δ
ubuntu-20.04_Python-3.10 80.62% <100.00%> (-0.22%) :arrow_down:
windows-2022_Python-3.10 ?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

codecov[bot] avatar May 08 '24 08:05 codecov[bot]