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

Add Anomaly alias #3832

Closed
wants to merge 6,060 commits into from
Closed

Add Anomaly alias #3832

wants to merge 6,060 commits into from

Conversation

ashwinvaidya17
Copy link
Contributor

Summary

  • As requested in CVS-147611

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

GalyaZalesskaya and others added 30 commits November 9, 2023 10:39
@GalyaZalesskaya
[release 1.5.0] Fix XAI algorithm for Detection (openvinotoolkit#2617)
d9cdcc2 
Update detection XAI algorithm
@yunchu
Update dependency constraint (openvinotoolkit#2622)
6462a4b
@yunchu
Update tpp (openvinotoolkit#2621)
6d3dd34
@goodsong81
Fix h-label bug of missing parent labels in output (openvinotoolkit#2626
2c9b745 
)

* Fix h-label bug of missing parent labels in output

* Fix h-label test data label schema

* Update CHANGELOG.md

---------
Signed-off-by: Songki Choi <songki.choi@intel.com>
@yunchu
Update publish workflow (openvinotoolkit#2625)
13a3993 
update publish workflow to push whl to internal pypi
@yunchu
Merge branch 'releases/1.5.0' into releases/1.4.0 for mergeback 1.4.4…
07491a0 
… to 1.5.0
@yunchu
bump datumaro version to ~=1.5.0
eadbb15
@yunchu
fixed mistake while mergeing back 1.4.4
516d395
@yunchu
modifiy readme
d84c3d4
@yunchu
remove openvino model wrapper class
5e5729b
@yunchu
remove openvino model wrapper tests
0325c14
@GalyaZalesskaya
[release 1.5.0] DeiT: enable tests + add ViTFeatureVectorHook (openvi…
16dd688 
…notoolkit#2630)

Add ViT feature vector hook
@goodsong81
Fix docs broken link to datatumaro_h-label
c461200 
Signed-off-by: Songki Choi <songki.choi@intel.com>
@goodsong81
Fix wrong label settings for non-anomaly task ModelAPIs
9f4a34b 
Signed-off-by: Songki Choi <songki.choi@intel.com>
@goodsong81
Merge branch 'releases/1.5.0' into mergeback/1.4.4-to-1.5.0
e459f1f
@yunchu
Merge pull request openvinotoolkit#2629 from openvinotoolkit/mergebac…
51ef6db 
…k/1.4.4-to-1.5.0

Mergeback 1.4.4 to releases/1.5.0
@yunchu
Update publish workflow for tag checking (openvinotoolkit#2632)
4ca63e8
@yunchu
Merge branch 'releases/1.5.0' into mergeback/1.5.0
e3bd52d
@GalyaZalesskaya
Update e2e tests for XAI Detection (openvinotoolkit#2634)
061aecf 
Fix e2e XAI ref value
@yunchu
Merge pull request openvinotoolkit#2633 from openvinotoolkit/mergebac…
104404b 
…k/1.5.0

Mergeback 1.5.0 to develop
@jaegukhyun
Disable QAT for newly added models (openvinotoolkit#2636)
9f591b6
@yunchu
Update release note and readme (openvinotoolkit#2637)
a2545f9 
* update release note and readme

* remove package upload step on internal publish wf
@yunchu
Merge branch 'releases/1.4.0' into mergeback/1.4.4rc2
5f1d0ba
@yunchu
update release note and, changelog, and readme
0e09a19
@yunchu
Merge branch 'releases/1.5.0' into mergeback/1.4.4rc2
8107372
@yunchu
Merge pull request openvinotoolkit#2639 from openvinotoolkit/mergebac…
9909d17 
…k/1.4.4rc2

Mergeback 1.4.4rc2 to 1.5.0
@yunchu
Merge branch 'releases/1.5.0' into mergeback/1.5.0
26d0fb4
@yunchu
update version string to 1.6.0dev
dd84a24
@yunchu
fix datumaro version to 1.6.0rc0
202c193
@yunchu @GalyaZalesskaya @jaegukhyun
Mergeback 1.5.0 to develop (openvinotoolkit#2642)
2f67686 
* Update publish workflow for tag checking (openvinotoolkit#2632)

* Update e2e tests for XAI Detection (openvinotoolkit#2634)

* Disable QAT for newly added models (openvinotoolkit#2636)

* Update release note and readme (openvinotoolkit#2637)

* remove package upload step on internal publish wf

* update release note and, changelog, and readme

* update version string to 1.6.0dev

---------

Co-authored-by: Galina Zalesskaya <galina.zalesskaya@intel.com>
Co-authored-by: Jaeguk Hyun <jaeguk.hyun@intel.com>
@ashwinvaidya17
Copy link
Contributor Author

ashwinvaidya17 commented Aug 21, 2024 •
edited
Loading

tox -e pre-commit-all-py310-pt1: This command reproduces exactly the same as CI: I pushed the change to the appropriate branch. ccba2e7 c261717

so now, let's see test

Thanks for pushing the change. This is what I had tried based on the actions file.

conda create -n temp python=3.10 -y
conda activate temp
pip install --require-hashes --no-deps -r requirements/gh-actions.txt
pip-compile --generate-hashes -o /tmp/otx-dev-requirements.txt requirements/dev.txt
pip install --require-hashes --no-deps -r /tmp/otx-dev-requirements.txt
rm /tmp/otx-dev-requirements.txt
tox -vv -e pre-commit-all-py310-pt1

But even with this I got the same result.

@ashwinvaidya17
Update tests
9c1c928 
Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>
@ashwinvaidya17
Merge branch 'ashwin/anomaly_geti_task_migration_1_6' of github.com:a…
d319610 
…shwinvaidya17/training_extensions into ashwin/anomaly_geti_task_migration_1_6
@ashwinvaidya17 ashwinvaidya17 mentioned this pull request Aug 21, 2024
Closed
8 tasks
@ashwinvaidya17
update tests
489a0b7 
Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>
@chuneuny-emily
Copy link
Contributor

@ashwinvaidya17 - OTX1.6.4 was already released into Geti 2.3.0 and develop branches. What is the Geti target version for this PR ?

@ashwinvaidya17
Copy link
Contributor Author

@ashwinvaidya17 - OTX1.6.4 was already released into Geti 2.3.0 and develop branches. What is the Geti target version for this PR ?

I am assuming the next version then. Maybe @A-Artemis can say more about this.

@A-Artemis
Copy link
Contributor

@ashwinvaidya17 - OTX1.6.4 was already released into Geti 2.3.0 and develop branches. What is the Geti target version for this PR ?

The Geti target is 2.5.0.

@chuneuny-emily
Copy link
Contributor

@A-Artemis @ashwinvaidya17 - It is difficult to merge the code changes targeting Geti 2.5 into the official OTX1.6.5 aimed at integrating Geti 2.3/2.4. For Geti 2.5, OTX2.x integration is scheduled. If an early drop is required due to preliminary work by other teams, it seems necessary to create a separate branch to check updated features. What do you think?

@chuneuny-emily
Copy link
Contributor

@ashwinvaidya17 - Please target this PR to OTX develop branch.
@A-Artemis - Please create a your own branch from Geti develop, then I'll integrate the latest OTX develop into the branch for your work.

@ashwinvaidya17
Copy link
Contributor Author

@chuneuny-emily This pr #3818 targets develop branch

@samet-akcay
Copy link
Contributor

As suggested by @chuneuny-emily, this is not needed and can be closed.

@samet-akcay samet-akcay reopened this Aug 27, 2024
@samet-akcay samet-akcay changed the base branch from releases/1.6.0 to develop August 27, 2024 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

@harimkang harimkang Awaiting requested review from harimkang

@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

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

@sungchul2 sungchul2 Awaiting requested review from sungchul2

@GalyaZalesskaya GalyaZalesskaya Awaiting requested review from GalyaZalesskaya

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

@goodsong81 goodsong81 Awaiting requested review from goodsong81

@yunchu yunchu Awaiting requested review from yunchu

@wonjuleee wonjuleee Awaiting requested review from wonjuleee

@eunwoosh eunwoosh Awaiting requested review from eunwoosh

Assignees
No one assigned
Labels
ALGO Any changes in OTX Algo Tasks implementation API Any changes in OTX API CLI Any changes in OTE CLI TEST Any changes in tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.