Commit 754b8a18 authored by Nick Felt's avatar Nick Felt Committed by TensorFlower Gardener
Browse files

Add 2.0 summary ops from TensorBoard to tf.summary via component package

This change has the effect of importing scalar(), image() and friends from TensorBoard's summary API into the TF 2.0 tf.summary module. It actually does so by replacing tf.summary entirely, via the component_api_helper() mechanism (in use for tf.estimator), with a replacement package from TensorBoard that re-exports the original tf.summary symbols, thereby creating a fused namespace.

The component package approach mitigates circular dependency issues (a risk because the TensorBoard ops in turn depend on TF) because the TF API __init__.py imports the core TF API symbols before it imports components, so if they in turn `import tensorflow` the module has been mostly initialized. It also means that TensorBoard needn't be importable at API generation time, only when TF is actually imported, which avoids issues like #22395.

Tracking issue: #25356.

PiperOrigin-RevId: 233874463
parent 737c9072
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment