Skip to content
This repository was archived by the owner on Jul 1, 2023. It is now read-only.

Fix Sequential Tests. #424

Merged
pschuh merged 1 commit into from
Aug 8, 2019
Merged

Fix Sequential Tests. #424

pschuh merged 1 commit into from
Aug 8, 2019

Conversation

@pschuh
Copy link
Contributor

@pschuh pschuh commented Aug 8, 2019

Fix bug introduced in #419.

@pschuh
Fix Sequential Tests.
b2bd043 
Fix bug introduced in tensorflow#419.
@pschuh pschuh requested a review from dan-zheng August 8, 2019 17:41
dan-zheng
dan-zheng approved these changes Aug 8, 2019
View reviewed changes
Copy link
Member

@dan-zheng dan-zheng left a comment

Choose a reason for hiding this comment

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

Thanks!

Context: calls to optimizer update functions were removed but the expected values were not updated.

@pschuh pschuh merged commit 1b77875 into tensorflow:master Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@dan-zheng dan-zheng dan-zheng approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pschuh @dan-zheng