Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@brianwa84
Copy link
Contributor

No description provided.

brianwa84 and others added 11 commits September 30, 2019 16:04
Quoting http://python-notes.curiousefficiency.org/en/latest/python_concepts/import_traps.html#the-double-import-trap:

"""
There?s a reason the general ?no package directories on sys.path? guideline
exists, and the fact that the interpreter itself doesn?t follow it when
determining sys.path[0] is the root cause of all sorts of grief.
"""

Also relevant: bazelbuild/bazel#7091

This is addressed toward issue tensorflow#539

PiperOrigin-RevId: 268058051
Sadly, they only trigger when a user actually uses some attribute of those
modules.

PiperOrigin-RevId: 268299350
…istribution docstring.

It falsely claimed that the function to transform a base distribution had to have a constant Jacobian. Replace it by a copy-paste from the docstring of tfd.ReparameterizationType.

PiperOrigin-RevId: 266475851
   - Add Linear/Polynomial kernel for testing. Fixed bug where shift parameter didn't have dimensions padded.
   - Added batch_shape/batch_shape_tensor() methods to KumaraswayTransformed.

PiperOrigin-RevId: 266190386
I scripted this change, which meant programmatically injecting imports. In order to satisfy linters, this also meant I needed to programmatically re-sort the imports. To get this right, I sed replaced instances of (e.g.) "tfd = tfp.distributions" with explicit imports.

PiperOrigin-RevId: 266444179
This CL updates remaining instances of `import tensorflow as tf` to the tf2-safe
style `import tensorflow.compat.v2 as tf`, and changes instances of
tf.compat.v2 to use a separate `import tensorflow.compat.v1 as tf1`

PiperOrigin-RevId: 266452786
…re is a bug in the generalized U turn criteria that biasing the sample.

Close tensorflow#542

PiperOrigin-RevId: 267211223
Fixed broken link for `Upgrading_from_Edward_to_Edward2.md`
@googlebot
Copy link

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

1 similar comment
@googlebot
Copy link

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added the cla: no Declares that the user has not signed CLA label Sep 30, 2019
@brianwa84 brianwa84 added the cla: yes Declares that the user has signed CLA label Sep 30, 2019
@googlebot
Copy link

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

ℹ️ Googlers: Go here for more info.

1 similar comment
@googlebot
Copy link

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

ℹ️ Googlers: Go here for more info.

@googlebot googlebot removed the cla: no Declares that the user has not signed CLA label Sep 30, 2019
@brianwa84
Copy link
Contributor Author

CLAs are good per #562 and #560

@brianwa84 brianwa84 requested a review from csuter September 30, 2019 20:31
@brianwa84 brianwa84 merged commit b696815 into tensorflow:r0.8 Oct 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Declares that the user has signed CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants