-
Notifications
You must be signed in to change notification settings - Fork 331
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
Reorganize [ops] and [custom_ops] #1010
Comments
Do you have any example? |
should live under ops on the other hand contains a ton of stuff (BoxMatcher, etc), but also IoU3D which is a custom TF op: https://github.com/keras-team/keras-cv/tree/master/keras_cv/ops |
Why tensorflow ops is here in the first place? Also it looks bazel is reuqired to build custom ops. It looks weird to see these in here as well. |
What do you mean? |
I mean, why do we need to write C code for some custom operations (ie. |
@innat Probably you missed these threads: |
Still a few utils to fix |
Currently, we have some tech debt here: ops contains some custom layers as well as some ops. We should really fix this and put the components that are like layers under layers instead of ops. Then we can merge custom_ops into the ops directory.
Its confusing to have both.
The text was updated successfully, but these errors were encountered: