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
Add support for operators RFFT, ComplexAbs #1114
Conversation
This pull request introduces 2 alerts when merging db08e76 into b4ac342 - view on LGTM.com new alerts:
|
Signed-off-by: xavier dupré <xavier.dupre@gmail.com>
Signed-off-by: xavier dupré <xavier.dupre@gmail.com>
Signed-off-by: xavier dupré <xavier.dupre@gmail.com>
tests/test_backend.py
Outdated
@@ -3624,6 +3629,27 @@ def test_conv2d_1_kernel_as_input(self): | |||
[1., 1., 4.]], dtype=np.float32).reshape(_KERNEL3x3) | |||
self._conv_kernel_as_input_test(x_val, w_val) | |||
|
|||
@unittest.skipIf(tf_signal is None, reason="TF does not have submodule signal.") |
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.
Can you make this check depend on the TF version instead of doing the whole try/catch import thing?
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.
Fixed.
Signed-off-by: xavier dupré <xavier.dupre@gmail.com>
Changes also validated with model https://tfhub.dev/google/nonsemantic-speech-benchmark/trill-distilled/3.