Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merge back 2.2.0.rc3 to develop #3963

Merged
merged 59 commits into from
Sep 25, 2024

Conversation

kprokofi
Copy link
Collaborator

Summary

How to test

Checklist

  • I have added unit tests to cover my changes.​
  • I have added integration tests to cover my changes.​
  • I have ran e2e tests and there is no issues.
  • I have added the description of my changes into CHANGELOG in my target branch (e.g., CHANGELOG in develop).​
  • I have updated the documentation in my target branch accordingly (e.g., documentation in develop).
  • I have linked related issues.

License

  • I submit my code changes under the same Apache 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: Apache-2.0

yunchu and others added 30 commits August 20, 2024 09:16
* Update test_cli.py

* Update tests/e2e/cli/test_cli.py

Co-authored-by: Eunwoo Shin <eunwoo.shin@intel.com>

* Update test_cli.py

* Update test_cli.py

---------

Co-authored-by: Eunwoo Shin <eunwoo.shin@intel.com>
* add docs about configurable input size

* update api usecase and fix bug
* Bump MAPI

* Update exportable code requirements
* change drop pixels value

* go safe, change only tested models

* minor
* Update step size for eff v2

* Update effb0 recipe
update maskrcnn swimt model type to transformer
…t#3903)

* add rtmpose_tiny for single obj

* add rtmpose_tiny for single obj

* modify test subset name

* fix unit test

* update recipe with reset
)

* Add warmup_iters for effv2

* Update max_epochs
* Fix pretrained_weight for timm

* Fix unit-test
* add rtmpose_tiny for single obj

* modify test subset name

* fix unit test

* property for pck
* Fix cached dir

* Pretrained weight download unit-test

* Fix pre-commit
* Restore output_raw_scores for classificaiton

* Add uts

* Fix linter
add label info to init

Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>
* Fix binary classification

* Add unit-tests
* ignore heads and disable smooth quant

* add activations_range_estimator_params

* update changelog
* Fix Task Chain

* Add multi-label case as well

* Add multi-label case as well2

* Add H-label case
…ies using label's id/key (openvinotoolkit#3932)

Modify label_groups for dm_label_categories with id/key of label
…oolkit#3933)

* remove datumaro attribute id from tiling

* add subset names
@kprokofi kprokofi enabled auto-merge September 20, 2024 15:55
@github-actions github-actions bot removed DEPENDENCY Any changes in any dependencies (new dep or its version) should be produced via Change Request on PM BUILD labels Sep 20, 2024
@harimkang harimkang changed the title Merge back 2.2 to develop Merge back 2.2.0.rc1 to develop Sep 23, 2024
@harimkang harimkang changed the title Merge back 2.2.0.rc1 to develop Merge back 2.2.0.rc2 to develop Sep 23, 2024
@harimkang
Copy link
Contributor

@eugene123tw Hi Eugene, Something is conflicting between 2.2.0.rc2 and develop with tiling, getting unit-test fails in s-seg tiling case (tests/unit/core/data/test_tiling.py::TestOTXTiling::test_tile_transform[SEMANTIC_SEGMENTATION]).
image

Can you take a look?

@eugene123tw
Copy link
Contributor

@kprokofi Could you add fixes in this PR? https://github.com/openvinotoolkit/training_extensions/pull/3973/files

Copy link

codecov bot commented Sep 23, 2024

Codecov Report

Attention: Patch coverage is 80.54299% with 43 lines in your changes missing coverage. Please review.

Project coverage is 81.61%. Comparing base (0e2ab6c) to head (97dad60).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
src/otx/core/data/dataset/anomaly.py 86.30% 10 Missing ⚠️
src/otx/core/metrics/pck.py 30.76% 9 Missing ⚠️
src/otx/core/model/anomaly.py 25.00% 6 Missing ⚠️
src/otx/algo/detection/rtdetr.py 40.00% 3 Missing ⚠️
src/otx/algo/segmentation/litehrnet.py 0.00% 3 Missing ⚠️
src/otx/core/data/pre_filtering.py 60.00% 2 Missing ⚠️
src/otx/core/model/keypoint_detection.py 33.33% 2 Missing ⚠️
src/otx/core/model/segmentation.py 0.00% 2 Missing ⚠️
src/otx/core/types/label.py 83.33% 2 Missing ⚠️
src/otx/algo/detection/yolox.py 66.66% 1 Missing ⚠️
... and 3 more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3963      +/-   ##
===========================================
- Coverage    81.70%   81.61%   -0.09%     
===========================================
  Files          289      289              
  Lines        27916    28047     +131     
===========================================
+ Hits         22808    22890      +82     
- Misses        5108     5157      +49     
Flag Coverage Δ
py310 81.61% <80.54%> (-0.07%) ⬇️
py311 ?

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@harimkang harimkang changed the title Merge back 2.2.0.rc2 to develop Merge back 2.2.0.rc3 to develop Sep 24, 2024
@sovrasov
Copy link
Contributor

@kprokofi could you provide a result of a local tests launch? CI doesn't work today

@kprokofi
Copy link
Collaborator Author

kprokofi commented Sep 25, 2024

detection tests are passed locally:
image

@sovrasov sovrasov disabled auto-merge September 25, 2024 11:24
@sovrasov sovrasov merged commit 3249377 into openvinotoolkit:develop Sep 25, 2024
19 of 22 checks passed
@kprokofi kprokofi deleted the kp/merge_back_2.2.0 branch September 25, 2024 11:25
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
DOC Improvements or additions to documentation TEST Any changes in tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.