New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[PR2/? from #774] Fix PhiloxRandom usage #785

Open

QuantumJaeYoo wants to merge 1 commit into tensorflow:master from QuantumJaeYoo:pr_2_fix_warnings

Contributor

QuantumJaeYoo commented Jun 16, 2023

This PR standardizes the way of using PhiloxRandom generators based on TensorFlow's random ops.

RandomOps have seed / seed2 attributes with SetIsStateful() option (codes)
Random generators are initialized at the constructor with OpContext (codes)
So the random number generator is a private member variable (codes)


          Fix PhiloxRandom usage

5c7873b

QuantumJaeYoo requested a review from MichaelBroughton

June 16, 2023 23:03

mhucka added the area/code-health label

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

area/code-health