Skip to content

Conversation

@seanpmorgan
Copy link
Member

Yikes... don't really have time to look into this more. What looks like a graph Tensor is running in eager execution as part of the keras test suite.

Constant breaks are getting draining here and it might just be easier to wait for 2.0-RC. I would say this layer is even a candidate for removal, but it continues to be used in academia and this bayesian inference paper may continue that:
https://arxiv.org/abs/1908.03491

@seanpmorgan seanpmorgan requested a review from facaiy as a code owner August 21, 2019 13:44
@WindQAQ WindQAQ self-requested a review August 21, 2019 14:08
Copy link
Member

@WindQAQ WindQAQ left a comment

Choose a reason for hiding this comment

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

Thanks!

@WindQAQ WindQAQ merged commit 28acaf4 into tensorflow:master Aug 21, 2019
@seanpmorgan seanpmorgan deleted the hotfix-layernorm branch September 4, 2019 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants