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

Merge back 1.2.1 RC1 & RC2 #2086

Merged
merged 17 commits into from
May 3, 2023
Merged

Merge back 1.2.1 RC1 & RC2 #2086

merged 17 commits into from
May 3, 2023

Conversation

goodsong81
Copy link
Contributor

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 added e2e tests for validation.
  • 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) 2023 Intel Corporation
# SPDX-License-Identifier: Apache-2.0

goodsong81 and others added 12 commits April 27, 2023 16:40
@goodsong81
Upgrade mmdeploy==0.14.0 from official PyPI (openvinotoolkit#2047)
1af3699 
Signed-off-by: Songki Choi <songki.choi@intel.com>
@eunwoosh @goodsong81
Bug fix: value of validation variable is changed after auto decrease …
b9ba513 
…batch size (openvinotoolkit#2053)

fix validate overriding
@supersoob @goodsong81
Integrate new ignored loss in semantic segmentation (openvinotoolkit#…
2447ef6 
…2065)
@supersoob @goodsong81
Remove unused modules in semantic segmentation (openvinotoolkit#2068)
1608b9f
@cih9088 @goodsong81
Add doc for fast data loading (openvinotoolkit#2069)
ea7fb12 
* docs: add fast data loading

* docs: add augmix

* docs: use reference

* docs: make first character of words capital

* docs: add simple example in cli command
@eunwoosh @goodsong81
Bug fix: set gpu_ids properly (openvinotoolkit#2071)
99b50b8 
use fisrt GPU of cuda visible devices
@eunwoosh @goodsong81
Bug fix: Progress goes 100% and back 0 % repeatedly during auto decre…
1b3ba85 
…ase bs in Geti (openvinotoolkit#2074)

* disable OTXProgressHook

* align with pre commit
@eugene123tw @goodsong81
Fix tiling 0 stride issue in parameter adapter (openvinotoolkit#2078)
57f1e67 
fix tiling 0 stride issue
@harimkang @goodsong81
Update instance-segmentation tutorial documentation (openvinotoolkit#…
09d0f83 
…2082)

* Update instance segmentation tutorial

* Fix some indent

* Update instance_segmentation.rst
@jaegukhyun @goodsong81
Optimize YOLOX data pipeline and add unit test for get_subset of Datu… (
6a65321 
openvinotoolkit#2075)

* Optimize YOLOX data pipeline and add unit test for get_subset of DatumaroAdapter

* Change tiling cache
@goodsong81
Release 1.2.1rc1
ed88a08
@goodsong81
Update demo otx version
917ca10 
Signed-off-by: Songki Choi <songki.choi@intel.com>
@github-actions github-actions bot added ALGO Any changes in OTX Algo Tasks implementation API Any changes in OTX API DOC Improvements or additions to documentation TEST Any changes in tests labels Apr 27, 2023
supersoob
supersoob previously approved these changes Apr 27, 2023
View reviewed changes
eunwoosh
eunwoosh previously approved these changes Apr 27, 2023
View reviewed changes
jaegukhyun
jaegukhyun previously approved these changes Apr 27, 2023
View reviewed changes
eugene123tw
eugene123tw previously approved these changes Apr 28, 2023
View reviewed changes
Copy link
Contributor

@eugene123tw eugene123tw left a comment

Choose a reason for hiding this comment

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

LGTM

By the way, could you add this PR to RC2? #2089

@goodsong81 goodsong81 marked this pull request as ready for review May 2, 2023 00:58
@goodsong81 goodsong81 requested a review from a team as a code owner May 2, 2023 00:58
@goodsong81
Copy link
Contributor Author

goodsong81 commented May 2, 2023 •
edited
Loading

LGTM

By the way, could you add this PR to RC2? #2089

I'll make another one for RC2 :)
-> Hmm, seg tests are failing. I'll include RC2 changes in this PR while dealing with the test failure.

@eugene123tw @goodsong81
Tiling Spatial Concatenation for OpenVINO IR (openvinotoolkit#2052)
8dfb50d 
* draft

* udpate pytest skip tag

* add spatial concatenation

* code quality fixes

* fix tests

* add unit test for ir scale factor

* revert pytest skip and address test failure in another PR

* refactor variable name
eugene123tw and others added 4 commits May 2, 2023 15:34
@eugene123tw @goodsong81
Add spatial concatenation to deployment demo (openvinotoolkit#2089)
1e48a40 
add spatial concatenation to deployment demo
@goodsong81
Merge remote-tracking branch 'upstream/develop' into songkich/merge-b…
554d093 
…ack-1.2.1rc1
@goodsong81
Release 1.2.1rc2
cf6994c 
Signed-off-by: Songki Choi <songki.choi@intel.com>
@goodsong81
Fix merge error in seg models
00963b5 
Signed-off-by: Songki Choi <songki.choi@intel.com>
@goodsong81 goodsong81 dismissed stale reviews from eugene123tw, jaegukhyun, eunwoosh, and supersoob via 00963b5 May 2, 2023 06:57
@goodsong81 goodsong81 changed the title Merge back 1.2.1 RC1 Merge back 1.2.1 RC1 & RC2 May 2, 2023
@goodsong81 goodsong81 merged commit a1f098d into openvinotoolkit:develop May 3, 2023
@goodsong81 goodsong81 deleted the songkich/merge-back-1.2.1rc1 branch May 3, 2023 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eunwoosh eunwoosh eunwoosh approved these changes

@jaegukhyun jaegukhyun jaegukhyun left review comments

@supersoob supersoob supersoob left review comments

@eugene123tw eugene123tw eugene123tw left review comments

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

@sungmanc sungmanc Awaiting requested review from sungmanc

Assignees
No one assigned
Labels
ALGO Any changes in OTX Algo Tasks implementation API Any changes in OTX API DOC Improvements or additions to documentation TEST Any changes in tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@goodsong81 @eugene123tw @supersoob @eunwoosh @jaegukhyun @cih9088 @harimkang