-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[PT FE] Added aten::logaddexp #28539
base: master
Are you sure you want to change the base?
Conversation
build_jenkins |
Can you let me know for any changes required/approve the CI request @rkazants Thanks! |
Co-authored-by: Roman Kazantsev <[email protected]>
Co-authored-by: Roman Kazantsev <[email protected]>
if self.dtype: | ||
x = x.to(self.dtype) | ||
y = y.to(self.dtype) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we don't need this casting. It is sufficient to generate input data of required type.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove and the map as well
|
||
@pytest.mark.nightly | ||
@pytest.mark.precommit | ||
@pytest.mark.parametrize( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please also add @pytest.mark.precommit_torch_export
. Take a look how other operations are enabled for torch.export case.
In order to test it locally, set env var PYTORCH_TRACING_MODE=EXPORT
: https://github.com/openvinotoolkit/openvino/blob/master/.github/workflows/job_pytorch_layer_tests.yml#L132
Details:
Added
aten::logcumsumexp
Tickets: