Skip to content

[CORE] Clamp pre post processing #29446

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
Merged
merged 24 commits into from
Mar 26, 2025
Merged

[CORE] Clamp pre post processing #29446

merged 24 commits into from
Mar 26, 2025

Conversation

@barnasm1
Copy link
Contributor

@barnasm1 barnasm1 commented Mar 13, 2025

Details:

  • add clamp operator for pre post processing

Tickets:

Diya910 and others added 10 commits February 13, 2025 12:28
@barnasm1 barnasm1 requested a review from a team as a code owner March 13, 2025 08:32
@barnasm1 barnasm1 changed the title Clamp pre post processing [CORE] Clamp pre post processing Mar 13, 2025
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings labels Mar 13, 2025
@barnasm1 barnasm1 self-assigned this Mar 13, 2025
praasz
praasz reviewed Mar 13, 2025
View reviewed changes
t-jankowski
t-jankowski reviewed Mar 13, 2025
View reviewed changes
@wine99 wine99 mentioned this pull request Mar 14, 2025
Merged
praasz
praasz approved these changes Mar 17, 2025
View reviewed changes
src/core/src/preprocess/preprocess_steps_impl.cpp
[min_value, max_value](const std::vector<Output<Node>>& nodes,
const std::shared_ptr<Model>& function,
PreprocessingContext& ctxt) {
OPENVINO_ASSERT(nodes.size() == 1,
Copy link
Contributor

@praasz praasz Mar 17, 2025

Choose a reason for hiding this comment

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

Add test case for this assert?

olpipi
olpipi reviewed Mar 25, 2025
View reviewed changes
Copy link
Contributor

@olpipi olpipi left a comment

Choose a reason for hiding this comment

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

Other pre/post PreProcessSteps have also python and c bindings. Do you want to add them in a separated PR?

@barnasm1 barnasm1 added this pull request to the merge queue Mar 26, 2025
github-merge-queue bot pushed a commit that referenced this pull request Mar 26, 2025
@barnasm1 @Diya910
@praasz @mlukasze
[CORE] Clamp pre post processing (#29446)
55e7451 
### Details:
- add clamp operator for pre post processing

### Tickets:
 - [*161080*](https://jira.devtools.intel.com/browse/CVS-161080)

---------

Co-authored-by: Diya910 <[email protected]>
Co-authored-by: Diya910 <[email protected]>
Co-authored-by: Pawel Raasz <[email protected]>
Co-authored-by: Michal Lukaszewski <[email protected]>
Merged via the queue into openvinotoolkit:master with commit ab21a9d Mar 26, 2025
188 checks passed
@barnasm1 barnasm1 deleted the clamp_pre_post_processing branch March 26, 2025 17:33
timxu826 pushed a commit to timxu826/openvino that referenced this pull request Apr 7, 2025
@barnasm1 @Diya910
@praasz @mlukasze
[CORE] Clamp pre post processing (openvinotoolkit#29446)
5b1f97a 
### Details:
- add clamp operator for pre post processing

### Tickets:
 - [*161080*](https://jira.devtools.intel.com/browse/CVS-161080)

---------

Co-authored-by: Diya910 <[email protected]>
Co-authored-by: Diya910 <[email protected]>
Co-authored-by: Pawel Raasz <[email protected]>
Co-authored-by: Michal Lukaszewski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@olpipi olpipi olpipi approved these changes

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

@praasz praasz praasz approved these changes

@maxnick maxnick Awaiting requested review from maxnick

@PatrikStepan PatrikStepan Awaiting requested review from PatrikStepan

Assignees

@barnasm1 barnasm1

Labels

category: Core OpenVINO Core (aka ngraph) category: CPP API OpenVINO CPP API bindings

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@barnasm1 @olpipi @t-jankowski @praasz @Diya910 @mlukasze