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

Fix tiling 0 stride issue in parameter adapter #2078

Conversation

eugene123tw
Copy link
Contributor

Summary

Fix tiling 0 stride issue in parameter adapter

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

@eugene123tw eugene123tw marked this pull request as ready for review April 26, 2023 10:00
@eugene123tw eugene123tw requested a review from a team as a code owner April 26, 2023 10:00
@github-actions github-actions bot added the ALGO Any changes in OTX Algo Tasks implementation label Apr 26, 2023
Copy link
Contributor

@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.

The logic looks OK to me. Thank you for the quick fix!

@goodsong81 goodsong81 added the FIX defect fix label Apr 26, 2023
@goodsong81 goodsong81 added this to the 1.2.0 milestone Apr 26, 2023
@goodsong81 goodsong81 merged commit 96c3557 into openvinotoolkit:releases/1.2.1 Apr 26, 2023
goodsong81 pushed a commit to goodsong81/training_extensions that referenced this pull request Apr 27, 2023
goodsong81 added a commit that referenced this pull request May 3, 2023
* Upgrade mmdeploy==0.14.0 from official PyPI (#2047)

* Bug fix: value of validation variable is changed after auto decrease batch size (#2053)

* Integrate new ignored loss in semantic segmentation (#2065)

* Remove unused modules in semantic segmentation (#2068)

* Add doc for fast data loading (#2069)

* Bug fix: set gpu_ids properly (#2071)

* Bug fix: Progress goes 100% and back 0 % repeatedly during auto decrease bs in Geti (#2074)

* Fix tiling 0 stride issue in parameter adapter (#2078)

* Update instance-segmentation tutorial documentation (#2082)

* Optimize YOLOX data pipeline and add unit test for get_subset of Datu… (#2075)

* Tiling Spatial Concatenation for OpenVINO IR (#2052)

* Add spatial concatenation to deployment demo (#2089)

---------

Signed-off-by: Songki Choi <songki.choi@intel.com>
Co-authored-by: Eunwoo Shin <eunwoo.shin@intel.com>
Co-authored-by: Soobee Lee <soobee.lee@intel.com>
Co-authored-by: Inhyuk Cho <andy.inhyuk.jo@intel.com>
Co-authored-by: Eugene Liu <eugene.liu@intel.com>
Co-authored-by: Harim Kang <harim.kang@intel.com>
Co-authored-by: Jaeguk Hyun <jaeguk.hyun@intel.com>
@eugene123tw eugene123tw deleted the eugene/CVS-109327-fix-tiling-adapter branch May 7, 2024 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ALGO Any changes in OTX Algo Tasks implementation FIX defect fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants