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

[core] Deprecate undefined type and use dynamic where possible #28766

Merged
merged 35 commits into from
Feb 20, 2025
Merged

[core] Deprecate undefined type and use dynamic where possible #28766

merged 35 commits into from
Feb 20, 2025

Conversation

praasz
Copy link
Contributor

@praasz praasz commented Jan 31, 2025 •
edited
Loading

Details:

  • Mark ov::element::undefined as deprecated
  • Use dynamic type instead of undefined

Related PRs:

Tickets:

@praasz
Deprecate undefined type and use dynamic where possible
5b5d6ec 
Signed-off-by: Pawel Raasz <pawel.raasz@intel.com>
@praasz praasz requested review from a team as code owners January 31, 2025 14:33
@praasz praasz requested review from ilya-lavrenov and itikhono and removed request for a team January 31, 2025 14:33
@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: Core OpenVINO Core (aka ngraph) category: IE Tests OpenVINO Test: plugins and common category: GPU OpenVINO GPU plugin labels Jan 31, 2025
@praasz
Copy link
Contributor Author

praasz commented Feb 12, 2025

@vladimir-paramuzov , @dmitry-gorokhov , @PatrikStepan Could have a look on plugins changes?

@github-actions github-actions bot removed the category: NPUW NPUW plugin label Feb 14, 2025
@mlukasze
Copy link
Contributor

@vladimir-paramuzov , @dmitry-gorokhov , @PatrikStepan

friendly reminder

@praasz
Restore style in GPU utils
352be14 
- remove code duplicate

Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
@praasz
Merge remote-tracking branch 'origin/master' into feature/use-element…
103b9a5 
…-dynamic-instead-undefined
@praasz praasz requested a review from p-durandin February 14, 2025 11:32
@akuporos akuporos mentioned this pull request Feb 17, 2025
Merged
@praasz
Merge branch 'master' into feature/use-element-dynamic-instead-undefined
83a8ce9
@praasz
Fix code style
42fd69e 
Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
@praasz
Merge remote-tracking branch 'origin/master' into feature/use-element…
bda3ad4 
…-dynamic-instead-undefined
@praasz
Minor changes
dc19f05 
Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
@praasz
Fix code style
b46d0db 
Signed-off-by: Raasz, Pawel <pawel.raasz@intel.com>
@praasz
Merge branch 'master' into feature/use-element-dynamic-instead-undefined
5a9cbe9
@t-jankowski t-jankowski added this pull request to the merge queue Feb 20, 2025
Merged via the queue into openvinotoolkit:master with commit d9c2aee Feb 20, 2025
192 checks passed
@t-jankowski t-jankowski deleted the feature/use-element-dynamic-instead-undefined branch February 20, 2025 14:55
@pereanub pereanub mentioned this pull request Feb 20, 2025
Merged
@pereanub
Copy link
Contributor

Some NPU regressions were added with this PR: Compilation fails for IntelDNS

@olpipi olpipi mentioned this pull request Feb 24, 2025
Closed
github-merge-queue bot pushed a commit that referenced this pull request Feb 25, 2025
@akuporos
[PyOV] Deprecate openvino.Type.undefined (#29020)
1fb0641 
### Details:
- to be merged after:
#28766

### Tickets:
 - [CVS-161723](https://jira.devtools.intel.com/browse/CVS-161723)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

@olpipi olpipi olpipi approved these changes

@t-jankowski t-jankowski t-jankowski approved these changes

@itikhono itikhono Awaiting requested review from itikhono itikhono is a code owner automatically assigned from openvinotoolkit/openvino-ie-transformations-maintainers

@zKulesza zKulesza Awaiting requested review from zKulesza zKulesza is a code owner automatically assigned from openvinotoolkit/openvino-docs-maintainers

@almilosz almilosz Awaiting requested review from almilosz

@akuporos akuporos Awaiting requested review from akuporos

@p-durandin p-durandin Awaiting requested review from p-durandin

Assignees

@ilya-lavrenov ilya-lavrenov

@dmitry-gorokhov dmitry-gorokhov

@PatrikStepan PatrikStepan

@vladimir-paramuzov vladimir-paramuzov

@slyalin slyalin

Labels
category: C API OpenVINO C API bindings category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings category: CPU OpenVINO CPU plugin category: docs_snippets OpenVINO docs snippets (docs/snippets) category: GPU OpenVINO GPU plugin category: IE Tests OpenVINO Test: plugins and common category: inference OpenVINO Runtime library - Inference category: IR FE OpenVINO IR v10 / v11 FrontEnd category: LP transformations OpenVINO Low Precision transformations category: NPU OpenVINO NPU plugin category: ONNX FE OpenVINO ONNX FrontEnd category: PDPD FE OpenVINO PaddlePaddle FrontEnd category: Python API OpenVINO Python bindings category: samples OpenVINO Runtime Samples category: TEMPLATE OpenVINO Template plugin category: TF FE OpenVINO TensorFlow FrontEnd category: TFL FE OpenVINO TensorFlow Lite FrontEnd category: transformations OpenVINO Runtime library - Transformations
Projects
None yet
Milestone
2025.1
Development

Successfully merging this pull request may close these issues.