| bc225bfaa534acc25047fe844f19edc333b7a76a |
|
14-Feb-2017 |
Peter Hawkins <phawkins@google.com> |
Fix code that ignores tensorflow::Status.
Add a new tensorflow::Status::IgnoreError() method to mark call sites where a Status has been intentionally ignored.
Change: 147402405
/external/tensorflow/tensorflow/core/framework/op_registration_test.cc
|
| e40bbecdfd1116468d162cf2ef8d4bb8f879c5e9 |
|
11-Jul-2016 |
Suharsh Sivakumar <suharshs@google.com> |
When an invalid op is registered, ensure that tf.load_op_library returns
a nice error rather than silently generating invalid python code.
Fixes:
- On linux, a fatal log that should be triggered in C++ seems to be ignored
causing the tf.load_op_library call to fail and exit.
We fix this by propagating the status from op registration to raise an
exception from python rather than print a fatal log.
The reason why the fatal log is being ignored on linux is unclear, my
theory is some sort of SWIG issue, but online research provided no results.
- Op registrations that fail are still added to the OpDef registration.
This is only an issue when the Fatal log is ignored, but we fix this
anyways.
Change: 127114085
/external/tensorflow/tensorflow/core/framework/op_registration_test.cc
|
| 84d123c6ffece5a31426c9b9eb48f04696045458 |
|
17-Jun-2016 |
Manjunath Kudlur <keveman@google.com> |
Made the OpRegistry::Register function take a factory instead of a
unique_ptr<OpRegistrationData>, so that the OpDef building can be
deferred until an call to Lookup.
Change: 125132276
/external/tensorflow/tensorflow/core/framework/op_registration_test.cc
|