Move utils to OTX common & adapter #1931

JihwanEom · 2023-03-23T09:45:11Z

Summary

Moved utils in MPA folder to OTX common and adapter.

Move all utils to otx.common & otx.adapter
Fix pre-commit issues
Update test codes

Unit tests

Classification integration tests

Summary

How to test

Checklist

I submit my changes into the develop branch
I have added description of my changes into CHANGELOG
I have updated the documentation accordingly
I have added tests to cover my changes
I have linked related issues

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) 2023 Intel Corporation
#
# SPDX-License-Identifier: MIT

Updated daily workflow - remove if statement to allow running on any branch by manually

re bugfix

Added the security related notification.

fix prettier issue

…raining_extensions into release/1.1.0

goodsong81

Thank you for your effort.

codecov-commenter · 2023-03-23T10:22:16Z

Codecov Report

Patch coverage: 81.81% and no project coverage change.

Comparison is base (72fc455) 80.75% compared to head (124fac5) 80.76%.

Additional details and impacted files

@@               Coverage Diff               @@
##           releases/1.1.0    #1931   +/-   ##
===============================================
  Coverage           80.75%   80.76%           
===============================================
  Files                 486      485    -1     
  Lines               33358    33348   -10     
===============================================
- Hits                26937    26932    -5     
+ Misses               6421     6416    -5

Impacted Files	Coverage Δ
otx/algorithms/common/utils/distance_utils.py	`0.00% <0.00%> (ø)`
otx/algorithms/common/utils/ext_loader.py	`0.00% <0.00%> (ø)`
otx/algorithms/common/utils/file.py	`100.00% <ø> (ø)`
otx/cli/manager/config_manager.py	`84.76% <0.00%> (ø)`
otx/algorithms/common/utils/task_adapt.py	`40.35% <9.09%> (ø)`
otx/algorithms/common/utils/mo_wrapper.py	`45.45% <41.66%> (ø)`
...orithms/common/adapters/mmcv/utils/config_utils.py	`77.94% <82.11%> (+3.29%)`	⬆️
otx/algorithms/common/utils/logger.py	`84.72% <88.88%> (ø)`
...sification/adapters/mmcls/datasets/otx_datasets.py	`76.86% <100.00%> (ø)`
...fication/adapters/mmcls/models/classifiers/byol.py	`89.24% <100.00%> (ø)`
... and 104 more

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

…raining_extensions into jeom/move-utils

* Bump up version to 1.1.0rc1 * Updated daily workflow (#1905) - remove if statement to allow running on any branch by manually * [FIX] Wrong test temp directory path (#1902) * Fix wrong test temp directory * Update tests/unit/algorithms/action/adapters/mmaction/utils/test_action_config_utils.py * Update test_action_config_utils.py * Update PR template (#1914) * Update . location. * Fix OTX1.1 -> Geti1.4 integration issues (#1910) * Add dill to requirements/api.txt * Return None instead of raising NotImplementedError in IMedia2DEntity.path * Move utils HPO uses into HPO directory (#1912) move utils into hpo dir * [FIX] Add stability to explain detection (#1901) Add stability to explain detection * [Fix] e2e tests FQ references (#1918) Fix FQ references * Move mpa.deploy to otx.algorithms.common (#1903) * Move deploy modules to otx * test: fix mmdeploy api replacement error * Fix pre-commit issues * Update otx/mpa/det/exporter.py * Update otx/mpa/exporter_mixin.py * Update otx/mpa/seg/exporter.py * Update otx/mpa/cls/exporter.py * Update otx/algorithms/common/adapters/mmdeploy/utils/mmdeploy.py * Update otx/algorithms/common/adapters/mmdeploy/utils/operations_domain.py * Add mmcls.VisionTransformer backbone support (#1908) * Add mmcls transformer backbones * Fix VisionTransformeroutput check * Add changes * Disable recording forward hooks in inferrer * Remove unused import * Move semantic-segmentation related codes to otx adapters (#1911) * Added security.md Add the security related notification * Update security.md fix prettier issue * Add custom exception class for CLI (#1919) * Add custom exception class for CLI * Fixed TCs for config_manager * Correct License to Apache * Remove temp hot key values (#1916) * Refactor OTX classification phase 1: move modules from MPA to OTX (#1893) * Revert TrainType typo (#1928) * Fix Conflict * Fix conflict * Fix unit-tests * Fix cli tests * Fix cli tests * Fix type * Move all hooks in MPA into OTX common mmcv adapter (#1922) * Add explanation for XAI & minor doc fixes (#1923) * [CI] Updated daily workflow (#1904) Updated daily workflow - remove if statement to allow running on any branch by manually * [FIX] re-bugfix: ATSS head loss (#1907) re bugfix * Fix typos * Explanation of Explanation * Add images & typo fixes * Fixes from comments * Add accuracy for OD explanation * Tutorial update * Add accuracy for BCCD and WGISD * Fix * Move utils to OTX common & adapter (#1931) * [CI] Updated daily workflow (#1904) Updated daily workflow - remove if statement to allow running on any branch by manually * [FIX] re-bugfix: ATSS head loss (#1907) re bugfix * Added security.md Added the security related notification. * Update security.md fix prettier issue * Move utils * Fix typo * Move classification unit tests to OTX tests (#1932) * [CI] Updated daily workflow (#1904) Updated daily workflow - remove if statement to allow running on any branch by manually * [FIX] re-bugfix: ATSS head loss (#1907) re bugfix * Added security.md Added the security related notification. * Update security.md fix prettier issue * Move classification unit tests * Apply mocker in RandAugment TC * Move otx.mpa.modules.ov -> otx.core.ov (#1929) * First Refactoring * Refactor mpa.modules.ov -> otx.core.ov * Remove otx.mpa.modules.ov * Fix docs & unit test paths * Fix some import * Add TODO comments * Add github icon link in doc head (#1934) * Fix invalid temp_dir inside unit tests (#1937) * Move all model utils and dataloaders/samplers in mpa to otx adapter (#1930) * Move stages to each algorithms (#1938) * Move stages to each algorithms * Change docs for action recognition (#1940) * Change docs for action recognition * Fix typo * Update MoViNet related parts * Add MoViNet performance * Revert table include Complexity and Model size * Fix tox to run tests on non-editable install * Fix packaging errors including cython module build / import issues (#1936) * Avoid cychon compile on Windows * Fix packaging errors * Fix pre-commit * Remove use_develop option from tox tests * Fix pre-commit * Fix tox -e -> -re for clean build * Fix import error * Add missing __init__.py * Fix relative -> absolute import * Revert to editable package in tox tests * Release 1.1.0 (#1933) * Release OTX 1.1.0 * Update known issues in CHANGELOG.md * Fix PR / issue # to links * Fix pre-commit * Bump-up version to 1.2 * Fix pre-commit --------- Signed-off-by: Songki Choi <songki.choi@intel.com> Co-authored-by: Yunchu Lee <yunchu.lee@intel.com> Co-authored-by: Harim Kang <harim.kang@intel.com> Co-authored-by: Eunwoo Shin <eunwoo.shin@intel.com> Co-authored-by: Emily Chun <emily.chun@intel.com> Co-authored-by: emily.chun <chuneuny@ikvensx012> Co-authored-by: Sungman Cho <sungman.cho@intel.com> Co-authored-by: Inhyuk Andy Cho <andy.inhyuk.jo@intel.com> Co-authored-by: Jaeguk Hyun <jaeguk.hyun@intel.com> Co-authored-by: Jihwan Eom <jihwan.eom@intel.com> Co-authored-by: Soobee Lee <soobee.lee@intel.com> Co-authored-by: Galina Zalesskaya <galina.zalesskaya@intel.com>

yunchu and others added 5 commits March 17, 2023 17:46

[CI] Updated daily workflow (#1904)

40559b0

Updated daily workflow - remove if statement to allow running on any branch by manually

[FIX] re-bugfix: ATSS head loss (#1907)

cd2519f

re bugfix

Added security.md

7aa577e

Added the security related notification.

Update security.md

11a8837

fix prettier issue

Merge branch 'releases/1.1.0' of https://github.com/openvinotoolkit/t…

6adb41f

…raining_extensions into release/1.1.0

github-actions bot added ALGO Any changes in OTX Algo Tasks implementation CLI Any changes in OTE CLI TEST Any changes in tests labels Mar 23, 2023

JihwanEom changed the title ~~Move modules to OTX common & adapter~~ Move utils to OTX common & adapter Mar 23, 2023

JihwanEom requested review from harimkang, jaegukhyun, eunwoosh, goodsong81 and supersoob March 23, 2023 09:53

JihwanEom marked this pull request as ready for review March 23, 2023 09:53

JihwanEom requested a review from a team as a code owner March 23, 2023 09:53

goodsong81 approved these changes Mar 23, 2023

View reviewed changes

sungmanc approved these changes Mar 23, 2023

View reviewed changes

sungmanc merged commit d9ec6ad into openvinotoolkit:releases/1.1.0 Mar 23, 2023

harimkang mentioned this pull request Mar 23, 2023

Move otx.mpa.modules.ov -> otx.core.ov #1929

Merged

11 tasks

JihwanEom added 3 commits March 24, 2023 01:32

Merge branch 'releases/1.1.0' of https://github.com/openvinotoolkit/t…

c9d51ca

…raining_extensions into jeom/move-utils

Move utils

b3ed8d7

Fix typo

124fac5

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Move utils to OTX common & adapter #1931

Move utils to OTX common & adapter #1931

JihwanEom commented Mar 23, 2023 •

edited

Loading

goodsong81 left a comment

codecov-commenter commented Mar 23, 2023

Move utils to OTX common & adapter #1931

Move utils to OTX common & adapter #1931

Conversation

JihwanEom commented Mar 23, 2023 • edited Loading

Summary

Unit tests

Classification integration tests

Summary

How to test

Checklist

License

goodsong81 left a comment

Choose a reason for hiding this comment

codecov-commenter commented Mar 23, 2023

Codecov Report

JihwanEom commented Mar 23, 2023 •

edited

Loading