-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[TF FE] Support AdjustHue operation #24769
[TF FE] Support AdjustHue operation #24769
Conversation
… value incorrect while others correct.
…/openvino into tf_fe/adjust_saturation_op
build_jenkins |
build_jenkins |
build_jenkins |
build_jenkins |
I think it will not be a big problem for a contributor to find it:) The main part has been already covered by you. Best regards, |
… into tf_fe/adjust_hue_op
Head branch was pushed to by a user without write access
@duydl, please do not rebase and do not change your code changes. All is fine with PR. Now we have infra problems and let us anticipate a fix to merge. Best regards, |
build_jenkins |
Sorry for wasting checks running. I know failed checks now not related to PR. Just fix one var in adjust_hue_op that still has "saturation" in naming. |
5ea0e96
into
openvinotoolkit:master
### Details: - Adapt from openvinotoolkit#24511 . Converting code repeated. Instead import from there? - Converting logic a little different from https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/kernels/image/adjust_hue_op.cc . Maybe less efficient? ### Tickets: - openvinotoolkit#24033 Sorry, commit history is wrong, should have created branch from latest upstream. --------- Co-authored-by: Roman Kazantsev <roman.kazantsev@intel.com>
Details:
Tickets:
Sorry, commit history is wrong, should have created branch from latest upstream.