-
Notifications
You must be signed in to change notification settings - Fork 137
Added support for the 'sign' op and its VJP. #144
Conversation
Could you add a test for each of these operators? |
I can do that yes. I just noticed that there were no tests for other wrappers of TF ops. It would be nice if we added a little helper that executes |
Yeah, other ops are mostly not tested. The migration made it so much easier to write tests now. |
I merged the |
Kokoro seems back online. Thanks @bgogul for fixing it! |
Co-Authored-By: Richard Wei <[email protected]>
Haha yeah CI servers do that sometimes and it's so frustrating when they just don't trigger :) |
Hmm I'm not sure what's wrong in Kokoro but earlier it was showing that same log while tests were passing locally. |
All tests pass locally on my machine. Merging. |
No description provided.