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

Fix to 'Model' object has no attribute '_container_nodes' error when using tf.keras.utils.plot_model(). #17668

Merged
merged 1 commit into from Mar 17, 2018

Conversation

sibyjackgrove
Copy link
Contributor

Fix to #17633. Duplicate of #17658
'Model' object has no attribute '_container_nodes' error when using tf.keras.utils.plot_model().
Replaced
if node_key in model._container_nodes:
with
if node_key in model._network_nodes: # pylint: disable=protected-access

in tensorflow\python\keras_impl\keras\utils\vis_utils.py.

…using tf.keras.utils.plot_model().

Fix to tensorflow#17633. Duplicate of tensorflow#17658
'Model' object has no attribute '_container_nodes' error when using tf.keras.utils.plot_model().
Replaced
if node_key in model._container_nodes:
with
if node_key in model._network_nodes: # pylint: disable=protected-access

in tensorflow\python\keras_impl\keras\utils\vis_utils.py.
@frankchn frankchn requested a review from fchollet March 13, 2018 04:34
@frankchn frankchn added kokoro:force-run Tests on submitted change awaiting review Pull request awaiting review labels Mar 13, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Tests on submitted change label Mar 13, 2018
Copy link
Member

@fchollet fchollet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fchollet fchollet merged commit 48178e0 into tensorflow:master Mar 17, 2018
StanislawAntol pushed a commit to StanislawAntol/tensorflow that referenced this pull request Mar 23, 2018
Fix to 'Model' object has no attribute '_container_nodes' error when using tf.keras.utils.plot_model().
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review Pull request awaiting review cla: yes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants