Addons: Re-name all C++ Ops to include package name

Created on 30 Jul 2019  路  7Comments  路  Source: tensorflow/addons

Most helpful comment

I'm putting together a pr to do this.

All 7 comments

Just copying a note I made in the RFC:
This is currently blocked as > is an invalid character in an Op Name at the moment

I believe this should be unblocked from the TF side now: https://github.com/tensorflow/tensorflow/commit/3bda03617967d3ff3c9af0e96c8a66b2bdad6ffc

(CC @ewilderj , FYI)

I'm putting together a pr to do this.

I just opened the PR. I had to make some changes to TF as part of this because there were still bugs in the TF RC1 w/ namespaces in op names, so this does not work w/ RC1. It works w/ the TF 2.0 nightly & should work w/ the upcoming RC2.

Closed as it's fixed by #516

Was this page helpful?
0 / 5 - 0 ratings