-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[Frontend][TFLite] support for FILL and SPLIT_V operators #5330
Conversation
@siju-samuel Thanks. Made the changes. |
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.
LGTM
@siju-samuel, Thanks. |
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.
Additionally I would like #5392 to be merged in so that we have some CI for these new operators which are newer than 1.13.1 which is what our CI is.
57bcbcd
to
ae89ef7
Compare
@u99127 I have merged the master and removed the duplicate code. |
@u99127, @siju-samuel, @FrozenGene, Could you please help in merging this PR. |
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.
LGTM, I can't do the merge myself.
Thanks @maheshambule @siju-samuel @u99127 merged now |
* tflite spliv ops * TFLITE fill and splitv ops * TFLITE fill and splitv ops * TFLITE fill and splitv ops * remove unnecessary operator check
* tflite spliv ops * TFLITE fill and splitv ops * TFLITE fill and splitv ops * TFLITE fill and splitv ops * remove unnecessary operator check
* tflite spliv ops * TFLITE fill and splitv ops * TFLITE fill and splitv ops * TFLITE fill and splitv ops * remove unnecessary operator check
@siju-samuel, @shoubhik, @kevinthesun Please help in review.