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

v1.4.1 #484

Merged
merged 8 commits into from
Dec 22, 2017
Merged

v1.4.1 #484

merged 8 commits into from
Dec 22, 2017

Conversation

rsepassi
Copy link
Contributor

No description provided.

Ryan Sepassi added 4 commits December 21, 2017 15:46
Add EarlyStoppingHook, PlateauOpHook, and MetricsBasedHook base class
87bfac5 
PiperOrigin-RevId: 179860572
Fix colab notebook
45a4b88 
PiperOrigin-RevId: 179871302
Pad eval batch to enable multi-device eval; skip T2TModel.top if T2TM…
b10286e 
…odel.body returns training loss

PiperOrigin-RevId: 179882031
Rm xrange usage from metrics_hook_test
83e5949 
PiperOrigin-RevId: 179882966
@rsepassi
Copy link
Contributor Author

Hold off on merging...working through the test failures.

python3 fix to metrics_hook_test
f2b620f 
PiperOrigin-RevId: 179886783
@rsepassi
Copy link
Contributor Author

Ok, all looks good for 1.4.1

@martinpopel
Copy link
Contributor

see #486 and #266 (comment) before merging

T2T Team added 2 commits December 22, 2017 09:03
Fix transformer's encode docstring.
ee947c9 
PiperOrigin-RevId: 179928442
Factor out common audio feature extraction and apply it to Librispeec…
2a07e8f 
…h dataset.

PiperOrigin-RevId: 179931584
@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and have the pull request author add another comment and the bot will run again. If the bot doesn't comment, it means it doesn't think anything has changed.

Add random seed, py3 fix, disable flaky test
02da1be 
PiperOrigin-RevId: 179942374
lukaszkaiser
lukaszkaiser approved these changes Dec 22, 2017
View reviewed changes
Copy link
Contributor

@lukaszkaiser lukaszkaiser left a comment

Choose a reason for hiding this comment

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

Thanks!

@lukaszkaiser lukaszkaiser merged commit c1cd875 into tensorflow:master Dec 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lukaszkaiser lukaszkaiser lukaszkaiser approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rsepassi @martinpopel @googlebot @lukaszkaiser