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.

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

Refactor to allow for a wider model in TIMM #3976

Merged
merged 53 commits into from
Sep 27, 2024

Conversation

harimkang
Copy link
Contributor

@harimkang harimkang commented Sep 24, 2024

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
* 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
* 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 (#3932)

Modify label_groups for dm_label_categories with id/key of label
* remove datumaro attribute id from tiling

* add subset names
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
@harimkang harimkang requested a review from sungchul2 September 26, 2024 04:51
@harimkang harimkang enabled auto-merge September 26, 2024 07:18
@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TEST Any changes in tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.