Skip to content

Support quantized::conv1d_relu #29730 #29819

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
Closed
wants to merge 11 commits into from
Closed

Support quantized::conv1d_relu #29730 #29819

wants to merge 11 commits into from

Conversation

Raghavyadav17
Copy link

@Raghavyadav17 Raghavyadav17 commented Mar 29, 2025 •
edited
Loading

Description
This PR adds support for the quantized::conv1d_relu operation in the PyTorch frontend of OpenVINO. It extends the existing implementation for quantized convolution layers to include conv1d_relu, ensuring compatibility and optimized inference.

Changes

  1. Implemented quantized::conv1d_relu in quantized_convnd.cpp.

  2. Ensured proper mapping and handling of quantized parameters.

Fixes #29730

@Raghavyadav17 Raghavyadav17 requested review from a team as code owners March 29, 2025 07:50
@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: GPU OpenVINO GPU plugin category: CPU OpenVINO CPU plugin labels Mar 29, 2025
@github-actions github-actions bot added category: Python API OpenVINO Python bindings category: C API OpenVINO C API bindings category: samples OpenVINO Runtime Samples category: ONNX FE OpenVINO ONNX FrontEnd category: dependency_changes Pull requests that update a dependency file category: CPP API OpenVINO CPP API bindings category: PyTorch FE OpenVINO PyTorch Frontend category: NPU OpenVINO NPU plugin labels Mar 29, 2025
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Mar 29, 2025
p-wysocki
p-wysocki requested changes Apr 4, 2025
View reviewed changes
Copy link
Contributor

@p-wysocki p-wysocki left a comment

Choose a reason for hiding this comment

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

Hello, please fix the PR - there has been some mistake in the commits, for example in the submodule versions.

@p-wysocki p-wysocki closed this Apr 11, 2025
@p-wysocki
Copy link
Contributor

Closed due to broken git and inactivity.

@Raghavyadav17
Copy link
Author

@p-wysocki will create a clean new PR for this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@p-wysocki p-wysocki p-wysocki requested changes

@slyalin slyalin Awaiting requested review from slyalin slyalin is a code owner automatically assigned from openvinotoolkit/openvino-pytorch-frontend-maintainers

@cavusmustafa cavusmustafa Awaiting requested review from cavusmustafa cavusmustafa is a code owner automatically assigned from openvinotoolkit/openvino-pytorch-frontend-maintainers

Assignees
No one assigned
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: dependency_changes Pull requests that update a dependency file category: GPU OpenVINO GPU plugin category: NPU OpenVINO NPU plugin category: ONNX FE OpenVINO ONNX FrontEnd category: Python API OpenVINO Python bindings category: PyTorch FE OpenVINO PyTorch Frontend category: samples OpenVINO Runtime Samples ExternalPR External contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Good First Issue]: Support quantized::conv1d_relu
4 participants
@Raghavyadav17 @p-wysocki @sys-openvino-ci @mlukasze