Skip to content

Mergeback 1.4.2rc5#2511

Merged
yunchu merged 7 commits intoopen-edge-platform:developfrom
yunchu:mergeback-1.4.2rc5
Sep 20, 2023
Merged

Mergeback 1.4.2rc5#2511
yunchu merged 7 commits intoopen-edge-platform:developfrom
yunchu:mergeback-1.4.2rc5

Conversation

@yunchu
Copy link
Copy Markdown
Contributor

@yunchu yunchu commented Sep 20, 2023

Summary

Mergeback PR for 1.4.2rc5

@sovrasov could you review this carefully for applying following changes into develop?
#2483
#2459

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

ashwinvaidya17 and others added 7 commits September 20, 2023 12:14
@ashwinvaidya17 @yunchu
Bug fix for albumentations (open-edge-platform#2467)
0d960de 
* bug fix for legacy openvino models

* Address albumentation issue

---------

Co-authored-by: Ashwin Vaidya <ashwinitinvaidya@gmail.com>
@yunchu
update for release 1.4.2rc3
689abff
@sovrasov @yunchu
Add a dummy hierarchical config required by MAPI (open-edge-platform#…
7c43e40 
…2483)
@yunchu
bump version to 1.4.2rc4
09ae4d7
@yunchu
Bump datumaro version (open-edge-platform#2502)
aac8ea6 
* bump datumaro version

* remove deprecated/reomved attribute usage of the datumaro
@sovrasov @yunchu
Upgrade nncf version for 1.4 release (open-edge-platform#2459)
3fcaf11 
* Upgrade nncf version

* Fix nncf interface warning

* Set the exact nncf version

* Update FQ refs after NNCF upgrade

* Use NNCF from pypi
@yunchu
Update version for release 1.4.2rc5 (open-edge-platform#2507)
d346f33
@yunchu yunchu requested a review from a team as a code owner September 20, 2023 04:43
@github-actions github-actions bot added DEPENDENCY Any changes in any dependencies (new dep or its version) should be produced via Change Request on PM API Any changes in OTX API ALGO Any changes in OTX Algo Tasks implementation TEST Any changes in tests DOC Improvements or additions to documentation labels Sep 20, 2023
@yunchu yunchu requested a review from sovrasov September 20, 2023 04:43
goodsong81
goodsong81 approved these changes Sep 20, 2023
View reviewed changes
Copy link
Copy Markdown

@goodsong81 goodsong81 left a comment

Choose a reason for hiding this comment

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

LGTM. Just 1 question below. Thanks!

sovrasov
sovrasov reviewed Sep 20, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@sovrasov sovrasov left a comment

Choose a reason for hiding this comment

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

LGTM from my side

@yunchu yunchu merged commit 12bf332 into open-edge-platform:develop Sep 20, 2023
@yunchu yunchu deleted the mergeback-1.4.2rc5 branch September 20, 2023 13:27
@sungmanc sungmanc mentioned this pull request Sep 21, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sovrasov sovrasov sovrasov approved these changes

+1 more reviewer

@goodsong81 goodsong81 goodsong81 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ALGO Any changes in OTX Algo Tasks implementation API Any changes in OTX API DEPENDENCY Any changes in any dependencies (new dep or its version) should be produced via Change Request on PM 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.

4 participants

@yunchu @goodsong81 @sovrasov @ashwinvaidya17