Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Refactor to allow for a wider model in TIMM #3976

Merged
merged 53 commits into from
Sep 27, 2024
Merged

Refactor to allow for a wider model in TIMM #3976

merged 53 commits into from
Sep 27, 2024

Conversation

harimkang
Copy link
Contributor

@harimkang harimkang commented Sep 24, 2024 •
edited
Loading

Summary

Refactor to allow for a wider model from timm

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
@yunchu
update for releases 2.2.0rc0
d1bd1d5
@harimkang
Fix Classification explain forward issue (#3867)
c16f985 
Fix bug
@chuneuny-emily @eunwoosh
Fix e2e code error (#3871)
cba5120 
* Update test_cli.py

* Update tests/e2e/cli/test_cli.py

Co-authored-by: Eunwoo Shin <[email protected]>

* Update test_cli.py

* Update test_cli.py

---------

Co-authored-by: Eunwoo Shin <[email protected]>
@eunwoosh
Add documentation about configurable input size (#3870)
b807c9d 
* add docs about configurable input size

* update api usecase and fix bug
@sungchul2
Fix zero-shot e2e (#3876)
2835aba 
Fix
@harimkang
Fix DeiT for multi-label classification (#3881)
ccf2d50 
Remove init_args
@harimkang
Fix Semi-SL for ViT accuracy drop (#3883)
e577b6a 
Remove init_args
@harimkang
Update docs for 2.2 (#3884)
d1dd2b0 
Update docs
@kprokofi
Fix mean and scale for segmentation task (#3885)
c17a923 
fix mean and scale
@sovrasov
Update MAPI in 2.2 (#3889)
d72feeb 
* Bump MAPI

* Update exportable code requirements
@kprokofi
Improve Semi-SL for LiteHRNet (small-medium case) (#3891)
00ed3a0 
* change drop pixels value

* go safe, change only tested models

* minor
@sooahleex
Improve h-cls for eff models (#3893)
2c6b4de 
* Update step size for eff v2

* Update effb0 recipe
@eugene123tw
Fix maskrcnn swin nncf acc drop (#3900)
0dc7a29 
update maskrcnn swimt model type to transformer
@wonjuleee
Add keypoint detection recipe for single object cases (#3903)
0d6799c 
* add rtmpose_tiny for single obj

* add rtmpose_tiny for single obj

* modify test subset name

* fix unit test

* update recipe with reset
@sooahleex
Improve acc drop of efficientnetv2 for h-label cls (#3907)
8115b52 
* Add warmup_iters for effv2

* Update max_epochs
@harimkang
Fix pretrained weight cached dir for timm (#3909)
4c8555e 
* Fix pretrained_weight for timm

* Fix unit-test
@wonjuleee
Fix keypoint detection single obj recipe (#3915)
52221e3 
* add rtmpose_tiny for single obj

* modify test subset name

* fix unit test

* property for pck
@harimkang
Fix cached dir for timm & hugging-face (#3914)
9265c59 
* Fix cached dir

* Pretrained weight download unit-test

* Fix pre-commit
@yunchu
Fix wrong template id mapping for anomaly (#3916)
5170736
@yunchu
Update script to allow setting otx version using env. variable (#3913)
f611cc1
@harimkang
Fix Datamodule creation for OV in AutoConfigurator (#3920)
425a479 
Fix datamodule for ov
@yunchu
Update tpp file for 2.2.0 (#3921)
7f1c7da
@kprokofi
Fix names for ignored scope [HOT-FIX, 2.2.0] (#3924)
51d1adf 
fix names for ignored scope
@sovrasov
Fix classification rt_info (#3922)
2bcf1b2 
* Restore output_raw_scores for classificaiton

* Add uts

* Fix linter
@ashwinvaidya17
Update label info (#3925)
112b2b2 
add label info to init

Signed-off-by: Ashwin Vaidya <[email protected]>
@harimkang
Fix binary classification metric task (#3928)
929132d 
* Fix binary classification

* Add unit-tests
@eugene123tw
Improve MaskRCNN SwinT NNCF (#3929)
706f99b 
* ignore heads and disable smooth quant

* add activations_range_estimator_params

* update changelog
@harimkang
Fix get_item for Chained Tasks in Classification (#3931)
53a7d9a 
* Fix Task Chain

* Add multi-label case as well

* Add multi-label case as well2

* Add H-label case
@sooahleex
Correct Keyerror for h-label cls in label_groups for dm_label_categor…
c3749e3 
…ies using label's id/key (#3932)

Modify label_groups for dm_label_categories with id/key of label
@eugene123tw
Remove datumaro attribute id from tiling, add subset names (#3933)
98a9cac 
* remove datumaro attribute id from tiling

* add subset names
@harimkang harimkang added this to the 2.3.0 milestone Sep 26, 2024
eunwoosh
eunwoosh reviewed Sep 26, 2024
View reviewed changes
Copy link
Contributor

@eunwoosh eunwoosh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your work. I left a comment. Please take a look.

src/otx/algo/classification/backbones/timm.py Show resolved Hide resolved
sungchul2
sungchul2 previously approved these changes Sep 26, 2024
View reviewed changes
@harimkang harimkang added this pull request to the merge queue Sep 26, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 26, 2024
@harimkang harimkang added this pull request to the merge queue Sep 26, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 26, 2024
@harimkang harimkang added this pull request to the merge queue Sep 27, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 27, 2024
@harimkang harimkang added this pull request to the merge queue Sep 27, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 27, 2024
@harimkang harimkang added this pull request to the merge queue Sep 27, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 27, 2024
@harimkang harimkang added this pull request to the merge queue Sep 27, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 27, 2024
@harimkang harimkang added this pull request to the merge queue Sep 27, 2024
@harimkang harimkang removed this pull request from the merge queue due to a manual request Sep 27, 2024
@harimkang harimkang merged commit 00d5604 into openvinotoolkit:develop Sep 27, 2024
19 of 20 checks passed
@harimkang harimkang mentioned this pull request Oct 8, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sungchul2 sungchul2 sungchul2 approved these changes

@eunwoosh eunwoosh eunwoosh approved these changes

@samet-akcay samet-akcay Awaiting requested review from samet-akcay samet-akcay is a code owner

@eugene123tw eugene123tw Awaiting requested review from eugene123tw eugene123tw is a code owner

@kprokofi kprokofi Awaiting requested review from kprokofi kprokofi is a code owner

@chuneuny-emily chuneuny-emily Awaiting requested review from chuneuny-emily chuneuny-emily is a code owner

@sovrasov sovrasov Awaiting requested review from sovrasov sovrasov is a code owner

@GalyaZalesskaya GalyaZalesskaya Awaiting requested review from GalyaZalesskaya GalyaZalesskaya is a code owner

@negvet negvet Awaiting requested review from negvet negvet is a code owner

@goodsong81 goodsong81 Awaiting requested review from goodsong81 goodsong81 is a code owner

@yunchu yunchu Awaiting requested review from yunchu yunchu is a code owner

@wonjuleee wonjuleee Awaiting requested review from wonjuleee wonjuleee is a code owner

Assignees
No one assigned
Labels
TEST Any changes in tests
Projects
None yet
Milestone
2.3.0
Development

Successfully merging this pull request may close these issues.