Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Branch 192771889 #18497

Merged
merged 110 commits into from Apr 13, 2018
Merged

Branch 192771889 #18497

merged 110 commits into from Apr 13, 2018

Conversation

qlzh727
Copy link
Member

@qlzh727 qlzh727 commented Apr 13, 2018

Third attempt to push to public.

Manually merged

tensorflow/contrib/data/python/ops/BUILD
tensorflow/python/framework/dtypes_test.py

tensorflower-gardener and others added 30 commits April 11, 2018 09:46
Explicitly track function input arg expansion into Placeholders,
and keep metadata to map between FunctionDef and GraphDef connectivity
formats.

PiperOrigin-RevId: 192462592
…les.get_or_create_global_step().

PiperOrigin-RevId: 192476077
PiperOrigin-RevId: 192479630
PiperOrigin-RevId: 192480452
…taset_op_test to "medium".

PiperOrigin-RevId: 192484895
PiperOrigin-RevId: 192491201
PiperOrigin-RevId: 192491335
PiperOrigin-RevId: 192498471
PiperOrigin-RevId: 192499401
…` in BN (not yet supported by TPU compilation).

PiperOrigin-RevId: 192502020
…. Choosing not to do this at static analysis because it exposes the scope to any node, making it easier to use by any specialization of a transformer.

PiperOrigin-RevId: 192502309
https://github.com/tensorflow/hub/tree/master/examples/image_retraining
has the same tool, upgraded to use TensorFlow Hub instead of raw graph defs.

PiperOrigin-RevId: 192502469
…_jacobian` methods to take event_ndims.

The class level event_ndims parameter is being deprecated in favor of passing it in
to the `log_det_jacobian` methods.

Specific changes:

  - `log_det_jacobian` signatures are now `log_det_jacobian(input, event_ndims)`

  - Constructors no long have event_ndims passed in (e.g. Affine() vs. Affine(event_ndims=0)).

  - All bijectors must specify a subset of [forward_min_event_ndims, inverse_min_event_ndims]. This is the minimal dimensionality the bijector operates on, with it being "broadcasted" to any passed in event_ndims (e.g. Exp has forward_min_event_ndims = 0. That means it operates on scalars. However, we can use the bijector on any event_ndims > 0 (i.e. we've broadcasted the transformation to work on any amount of event_ndims > 0), and jacobian reduction will work in those cases.

As a result of this change, all bijectors should "broadcast" (e.g. Sigmoid now works on any number of event_ndims).

Other changes (internal and documentation):
  - Added clarifications on Jacobian Determinant vs. Jacobian Matrix.
  - Added clarifications on min_event_ndims, and what the jacobian reduction is over.
  - Changed caching of ildj to be keyed on event_ndims.
  - Several bug fixes to bugs unearthed while writing this code (e.g. transformed distribution shape computation being incorrect)

PiperOrigin-RevId: 192504919
…n by mutating

NodeDef before creating the copied operation.

PiperOrigin-RevId: 192505209
…e updated checkpoints from the contents.

PiperOrigin-RevId: 192517819
…able in the metagraph.

PiperOrigin-RevId: 192518307
… trees estimators.

PiperOrigin-RevId: 192521398
data are saved with host prefix.

PiperOrigin-RevId: 192523668
Implement a vectorized way to compute the same thing instead.

PiperOrigin-RevId: 192524667
fchollet and others added 19 commits April 12, 2018 19:04
…r,Platform}.

These types live inside StreamExecutor's namespace, but they are
specific to XLA.  Therefore they either shouldn't live in SE's namespace
or should have "XLA" in the name.

Moving them out of SE's namespace is ugly, because almost every type
used inside of these headers then needs to be qualified.  So name-change
it is.

This patch was generated by a mechanical find/replace.

PiperOrigin-RevId: 192724238
-- Refactor utility functions into pruning_utils.py and add tests

PiperOrigin-RevId: 192727737
PiperOrigin-RevId: 192728635
PiperOrigin-RevId: 192756152
PiperOrigin-RevId: 192765203
PiperOrigin-RevId: 192768744
…ce. This removes the need to explicitly import internal components (barring the tf module which cannot be imported directly).

PiperOrigin-RevId: 192771440
@qlzh727 qlzh727 added the kokoro:force-run Tests on submitted change label Apr 13, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Apr 13, 2018
@qlzh727 qlzh727 added the kokoro:force-run Tests on submitted change label Apr 13, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Apr 13, 2018
@qlzh727
Copy link
Member Author

qlzh727 commented Apr 13, 2018

XLA test is broken by internal env change, which has nothing to do with this PR. I think this is good to go now.

@qlzh727 qlzh727 merged commit cda36b8 into tensorflow:master Apr 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet