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

Fix problem_0_steps in checkpoint averages #1273

Merged
merged 2 commits into from
Dec 5, 2018

Conversation

senarvi
Copy link

@senarvi senarvi commented Dec 4, 2018

Currently t2t-avg-all creates an average of the train_stats/problem_0_steps variable, of type float32, which prevents restoring variables from the averaged checkpoints. This fix will create a dummy train_stats/problem_0_steps variable, of type int32.

@googlebot googlebot added the cla: yes PR author has signed CLA label Dec 4, 2018
@senarvi
Copy link
Author

senarvi commented Dec 4, 2018

For some reason the CI system was unable to install the code on Python 3.6.

@afrozenator
Copy link
Contributor

Thanks @senarvi !

@afrozenator afrozenator merged commit 6d572c3 into tensorflow:master Dec 5, 2018
theorm added a commit to theorm/tensor2tensor that referenced this pull request Dec 5, 2018
* master:
  internal merge of PR tensorflow#1274
  add a metrics for calculating pearson correlation coefficient (tensorflow#1274)
  Rm dataset
  internal merge of PR tensorflow#1269
  Improved model free eval. (tensorflow#1269)
  Re-enable eval summaries in the correct directory
  Create an integer problem_0_steps variable. (tensorflow#1273)
  Small En-Ro dataset for low-resource experiments, use that one in translation multi-problem; and a few corrections.
  Rm evaluation_hooks summarysaver. This shouldn't be here. Directory may be eval_train. Estimator is already logging summaries for eval.
  Add support for multiproblem to mtf_model
  add explicit taskid check for all multiproblem_weight functions
  Readd nfg to open-source T2T for external collaboration.
  Starting work on a new mesh_tensorflow Transformer implementation.  This implementation lives in the mesh_tensorflow library and does not depend on Tensor2Tensor.
kpe pushed a commit to kpe/tensor2tensor that referenced this pull request Mar 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has signed CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants