Swap Estimator V1 and V2 so that Estimator V1 contains all of the old method definitions.

Previously any new methods and modifications added to EstimatorV2 would also get attached to EstimatorV1 (because EstimatorV1 inherited from V2). This would have become problematic as development on EstimatorV2 continued, because compat V1 behavior/API should remain the same.

PiperOrigin-RevId: 242166024
This commit is contained in:
Katherine Wu 2019-04-05 11:55:50 -07:00 committed by TensorFlower Gardener
parent 878c50a904
commit 86afd7efc8
34 changed files with 19 additions and 23 deletions
tensorflow
python/tpu
tools/api/golden
v1
v2

View File

@ -2403,7 +2403,8 @@ class TPUEstimator(estimator_lib.Estimator):
save_variables=True,
mode=model_fn_lib.ModeKeys.PREDICT,
export_tags=None,
check_variables=True):
check_variables=True,
strip_default_attrs=True):
if self._export_to_tpu and mode != model_fn_lib.ModeKeys.PREDICT:
logging.warning('TPUEstimator only handles mode PREDICT for exporting '
'when `export_to_tpu` is `True`; Mode {} will be ignored '
@ -2420,7 +2421,8 @@ class TPUEstimator(estimator_lib.Estimator):
save_variables,
mode=mode,
export_tags=export_tags,
check_variables=check_variables))
check_variables=check_variables,
strip_default_attrs=strip_default_attrs))
if self._export_to_tpu and mode == model_fn_lib.ModeKeys.PREDICT:
input_receiver_fn_map = {
@ -2441,7 +2443,8 @@ class TPUEstimator(estimator_lib.Estimator):
save_variables=save_variables,
mode=mode,
export_tags=export_tags,
check_variables=check_variables))
check_variables=check_variables,
strip_default_attrs=strip_default_attrs))
def _call_model_fn(self, features, labels, mode, config):
if mode == _REWRITE_FOR_INFERENCE_MODE:

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.BaselineClassifier"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.baseline.BaselineClassifier\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.BaselineEstimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.baseline.BaselineEstimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.BaselineRegressor"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.baseline.BaselineRegressor\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -3,7 +3,6 @@ tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees.BoostedTreesClassifier\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees._BoostedTreesBase\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -3,7 +3,6 @@ tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees.BoostedTreesEstimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees._BoostedTreesBase\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -3,7 +3,6 @@ tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees.BoostedTreesRegressor\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees._BoostedTreesBase\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.DNNClassifier"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn.DNNClassifier\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.DNNEstimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn.DNNEstimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.DNNLinearCombinedClassifier"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn_linear_combined.DNNLinearCombinedClassifier\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.DNNLinearCombinedEstimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn_linear_combined.DNNLinearCombinedEstimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.DNNLinearCombinedRegressor"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn_linear_combined.DNNLinearCombinedRegressor\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.DNNRegressor"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn.DNNRegressor\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -1,7 +1,6 @@
path: "tensorflow.estimator.Estimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.LinearClassifier"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.linear.LinearClassifier\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.LinearEstimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.linear.LinearEstimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.LinearRegressor"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.linear.LinearRegressor\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,7 +2,6 @@ path: "tensorflow.estimator.experimental.KMeans"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.kmeans.KMeansClustering\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "ALL_DISTANCES"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.BaselineClassifier"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.baseline.BaselineClassifierV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.BaselineEstimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.baseline.BaselineEstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.BaselineRegressor"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.baseline.BaselineRegressorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -3,7 +3,6 @@ tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees.BoostedTreesClassifier\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees._BoostedTreesBase\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -3,7 +3,6 @@ tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees.BoostedTreesEstimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees._BoostedTreesBase\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -3,7 +3,6 @@ tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees.BoostedTreesRegressor\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.boosted_trees._BoostedTreesBase\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.DNNClassifier"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn.DNNClassifierV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.DNNEstimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn.DNNEstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.DNNLinearCombinedClassifier"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn_linear_combined.DNNLinearCombinedClassifierV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.DNNLinearCombinedEstimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn_linear_combined.DNNLinearCombinedEstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.DNNLinearCombinedRegressor"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn_linear_combined.DNNLinearCombinedRegressorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.DNNRegressor"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.dnn.DNNRegressorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -1,6 +1,7 @@
path: "tensorflow.estimator.Estimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.LinearClassifier"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.linear.LinearClassifierV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.LinearEstimator"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.linear.LinearEstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"

View File

@ -2,6 +2,7 @@ path: "tensorflow.estimator.LinearRegressor"
tf_class {
is_instance: "<class \'tensorflow_estimator.python.estimator.canned.linear.LinearRegressorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.EstimatorV2\'>"
is_instance: "<class \'tensorflow_estimator.python.estimator.estimator.Estimator\'>"
is_instance: "<type \'object\'>"
member {
name: "config"