tensorflow / quantum

Hybrid Quantum-Classical Machine Learning in TensorFlow
https://www.tensorflow.org/quantum
Apache License 2.0
1.77k stars 560 forks source link

Replace tensorflow::Status::OK() with tensorflow::{Status, OkStatus}(). #719

Closed jblespiau closed 1 year ago

jblespiau commented 1 year ago

This is the equivalent of the internal cl/477929395

jaeyoo commented 1 year ago

LGTM, but I think this change requires bumping up to the latest tensorflow in WORKSPACE & setup, etc. WDYT @MichaelBroughton ?

After discussion offline with PR author, confirmed that it's backward compatible. approved.

jaeyoo commented 1 year ago

https://github.com/tensorflow/quantum/pull/749 solves this PR together. closed.