Add OpenVINO backend support for numpy.expm1 #21127
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #21127 +/- ##
==========================================
- Coverage 82.69% 82.67% -0.02%
==========================================
Files 564 564
Lines 54132 54138 +6
Branches 8411 8412 +1
==========================================
- Hits 44765 44761 -4
- Misses 7294 7302 +8
- Partials 2073 2075 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
keras/src/backend/openvino/numpy.py
Outdated
| @@ -1379,3 +1379,12 @@ def argpartition(x, kth, axis=-1): | |||
| raise NotImplementedError( | |||
| "`argpartition` is not supported with openvino backend" | |||
| ) | |||
|
|
|||
|
|
|||
| def expm1(x): | |||
There was a problem hiding this comment.
there exists expm1 declaration for which you need provide implementation. No need to create a new declaration.
Also, please activate test for this op
There was a problem hiding this comment.
implemented to its previous declaration, what exactly do you mean to activate test for this op?
There was a problem hiding this comment.
check other PRs how they remove
| ) | ||
| ) |
| element_type = None | ||
| if isinstance(x, OpenVINOKerasTensor): | ||
| element_type = x.output.get_element_type() |
| element_type = None | ||
| if isinstance(x, OpenVINOKerasTensor): | ||
| element_type = x.output.get_element_type() | ||
| x = get_ov_output(x, element_type) |
There was a problem hiding this comment.
just leave: x = get_ov_output(x)
This pull request adds OpenVINO backend support for numpy.expm1. The expm1 function is implemented using OpenVINO operations to perform element-wise exp(x) - 1.
Changes include:
numpy.pyfor OpenVINO backend.