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.

Sign up for GitHub

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

Jump to bottom

Removed no longer supported call in_eager_execution() #23560

Merged
merged 1 commit into from
Nov 7, 2018
Merged

Removed no longer supported call in_eager_execution() #23560

merged 1 commit into from
Nov 7, 2018

Commits on Nov 6, 2018

  1. Removed no longer supported call to in_eager_execution 

    changes to model_analyser.analyse(...):
- Swapped context.in_eager_execution() to the currently supported context.executing_eagerly().
- Added negation to eager check. In all likelihood, the negation was always supposed to be there since getting default graph in eager mode does not make sense. The current `if` condition is likely a bug. The proposed fix is consistent with other functions in this module, e.g., `profile(...)`, line 339.
    @gehring
    gehring committed Nov 6, 2018
    Configuration menu
    Copy the full SHA
    c9b4689 View commit details
    Browse the repository at this point in the history