Consistently import all estimator symbols via tensorflow_estimator as opposed to using tensorflow.estimator for some symbols.

PiperOrigin-RevId: 443417581
This commit is contained in:
Fabien Hertschuh 2022-04-21 10:36:28 -07:00 committed by A. Unique TensorFlower
parent de585e5619
commit d47cc695cd
11 changed files with 21 additions and 33 deletions

View file

@ -27,7 +27,7 @@ py_library(
"binary_class_head.py", "binary_class_head.py",
], ],
srcs_version = "PY3", srcs_version = "PY3",
deps = ["//third_party/py/tensorflow:tensorflow_estimator"], deps = ["//third_party/tensorflow_estimator/python/estimator:binary_class_head"],
) )
py_library( py_library(
@ -36,7 +36,7 @@ py_library(
"multi_class_head.py", "multi_class_head.py",
], ],
srcs_version = "PY3", srcs_version = "PY3",
deps = ["//third_party/py/tensorflow:tensorflow_estimator"], deps = ["//third_party/tensorflow_estimator/python/estimator:multi_class_head"],
) )
py_library( py_library(
@ -45,7 +45,7 @@ py_library(
"multi_label_head.py", "multi_label_head.py",
], ],
srcs_version = "PY3", srcs_version = "PY3",
deps = ["//third_party/py/tensorflow:tensorflow_estimator"], deps = ["//third_party/tensorflow_estimator/python/estimator:multi_label_head"],
) )
py_library( py_library(
@ -54,10 +54,7 @@ py_library(
"dnn.py", "dnn.py",
], ],
srcs_version = "PY3", srcs_version = "PY3",
deps = [ deps = [":head_utils"],
":head_utils",
"//third_party/py/tensorflow:tensorflow_estimator",
],
) )
py_library( py_library(
@ -78,7 +75,6 @@ py_test(
":binary_class_head", ":binary_class_head",
":test_utils", ":test_utils",
"//tensorflow_privacy/privacy/optimizers:dp_optimizer_keras", "//tensorflow_privacy/privacy/optimizers:dp_optimizer_keras",
"//third_party/py/tensorflow:tensorflow_estimator",
], ],
) )
@ -92,7 +88,6 @@ py_test(
":multi_class_head", ":multi_class_head",
":test_utils", ":test_utils",
"//tensorflow_privacy/privacy/optimizers:dp_optimizer_keras", "//tensorflow_privacy/privacy/optimizers:dp_optimizer_keras",
"//third_party/py/tensorflow:tensorflow_estimator",
], ],
) )
@ -106,7 +101,6 @@ py_test(
":multi_label_head", ":multi_label_head",
":test_utils", ":test_utils",
"//tensorflow_privacy/privacy/optimizers:dp_optimizer_keras", "//tensorflow_privacy/privacy/optimizers:dp_optimizer_keras",
"//third_party/py/tensorflow:tensorflow_estimator",
], ],
) )

View file

@ -14,16 +14,16 @@
"""Binary class head for Estimator that allow integration with TF Privacy.""" """Binary class head for Estimator that allow integration with TF Privacy."""
import tensorflow as tf import tensorflow as tf
from tensorflow import estimator as tf_estimator
from tensorflow.python.keras.utils import losses_utils # pylint: disable=g-direct-tensorflow-import from tensorflow.python.keras.utils import losses_utils # pylint: disable=g-direct-tensorflow-import
from tensorflow_estimator.python.estimator import model_fn from tensorflow_estimator.python.estimator import model_fn
from tensorflow_estimator.python.estimator.canned import prediction_keys from tensorflow_estimator.python.estimator.canned import prediction_keys
from tensorflow_estimator.python.estimator.export import export_output from tensorflow_estimator.python.estimator.export import export_output
from tensorflow_estimator.python.estimator.head import base_head from tensorflow_estimator.python.estimator.head import base_head
from tensorflow_estimator.python.estimator.head import binary_class_head
from tensorflow_estimator.python.estimator.mode_keys import ModeKeys from tensorflow_estimator.python.estimator.mode_keys import ModeKeys
class DPBinaryClassHead(tf_estimator.BinaryClassHead): class DPBinaryClassHead(binary_class_head.BinaryClassHead):
"""Creates a TF Privacy-enabled version of BinaryClassHead.""" """Creates a TF Privacy-enabled version of BinaryClassHead."""
def __init__(self, def __init__(self,

View file

@ -14,10 +14,10 @@
import numpy as np import numpy as np
import tensorflow as tf import tensorflow as tf
from tensorflow import estimator as tf_estimator
from tensorflow_privacy.privacy.estimators import binary_class_head from tensorflow_privacy.privacy.estimators import binary_class_head
from tensorflow_privacy.privacy.estimators import test_utils from tensorflow_privacy.privacy.estimators import test_utils
from tensorflow_privacy.privacy.optimizers.dp_optimizer_keras import DPKerasSGDOptimizer from tensorflow_privacy.privacy.optimizers.dp_optimizer_keras import DPKerasSGDOptimizer
from tensorflow_estimator.python.estimator import estimator
class DPBinaryClassHeadTest(tf.test.TestCase): class DPBinaryClassHeadTest(tf.test.TestCase):
@ -59,7 +59,7 @@ class DPBinaryClassHeadTest(tf.test.TestCase):
noise_multiplier=0.0, noise_multiplier=0.0,
num_microbatches=2) num_microbatches=2)
model_fn = test_utils.make_model_fn(head, optimizer, feature_columns) model_fn = test_utils.make_model_fn(head, optimizer, feature_columns)
classifier = tf_estimator.Estimator(model_fn=model_fn) classifier = estimator.Estimator(model_fn=model_fn)
classifier.train( classifier.train(
input_fn=test_utils.make_input_fn(train_features, train_labels, True), input_fn=test_utils.make_input_fn(train_features, train_labels, True),

View file

@ -15,13 +15,12 @@
import tensorflow as tf import tensorflow as tf
from tensorflow import estimator as tf_estimator
from tensorflow_privacy.privacy.estimators import head_utils from tensorflow_privacy.privacy.estimators import head_utils
from tensorflow_estimator.python.estimator import estimator from tensorflow_estimator.python.estimator import estimator
from tensorflow_estimator.python.estimator.canned import dnn from tensorflow_estimator.python.estimator.canned import dnn
class DNNClassifier(tf_estimator.Estimator): class DNNClassifier(estimator.Estimator):
"""DP version of `tf.estimator.DNNClassifier`.""" """DP version of `tf.estimator.DNNClassifier`."""
def __init__( def __init__(

View file

@ -14,16 +14,16 @@
"""Multiclass head for Estimator that allow integration with TF Privacy.""" """Multiclass head for Estimator that allow integration with TF Privacy."""
import tensorflow as tf import tensorflow as tf
from tensorflow import estimator as tf_estimator
from tensorflow.python.keras.utils import losses_utils # pylint: disable=g-direct-tensorflow-import from tensorflow.python.keras.utils import losses_utils # pylint: disable=g-direct-tensorflow-import
from tensorflow_estimator.python.estimator import model_fn from tensorflow_estimator.python.estimator import model_fn
from tensorflow_estimator.python.estimator.canned import prediction_keys from tensorflow_estimator.python.estimator.canned import prediction_keys
from tensorflow_estimator.python.estimator.export import export_output from tensorflow_estimator.python.estimator.export import export_output
from tensorflow_estimator.python.estimator.head import base_head from tensorflow_estimator.python.estimator.head import base_head
from tensorflow_estimator.python.estimator.head import multi_class_head
from tensorflow_estimator.python.estimator.mode_keys import ModeKeys from tensorflow_estimator.python.estimator.mode_keys import ModeKeys
class DPMultiClassHead(tf_estimator.MultiClassHead): class DPMultiClassHead(multi_class_head.MultiClassHead):
"""Creates a TF Privacy-enabled version of MultiClassHead.""" """Creates a TF Privacy-enabled version of MultiClassHead."""
def __init__(self, def __init__(self,

View file

@ -14,10 +14,10 @@
import numpy as np import numpy as np
import tensorflow as tf import tensorflow as tf
from tensorflow import estimator as tf_estimator
from tensorflow_privacy.privacy.estimators import multi_class_head from tensorflow_privacy.privacy.estimators import multi_class_head
from tensorflow_privacy.privacy.estimators import test_utils from tensorflow_privacy.privacy.estimators import test_utils
from tensorflow_privacy.privacy.optimizers.dp_optimizer_keras import DPKerasSGDOptimizer from tensorflow_privacy.privacy.optimizers.dp_optimizer_keras import DPKerasSGDOptimizer
from tensorflow_estimator.python.estimator import estimator
class DPMultiClassHeadTest(tf.test.TestCase): class DPMultiClassHeadTest(tf.test.TestCase):
@ -63,7 +63,7 @@ class DPMultiClassHeadTest(tf.test.TestCase):
noise_multiplier=0.0, noise_multiplier=0.0,
num_microbatches=2) num_microbatches=2)
model_fn = test_utils.make_model_fn(head, optimizer, feature_columns) model_fn = test_utils.make_model_fn(head, optimizer, feature_columns)
classifier = tf_estimator.Estimator(model_fn=model_fn) classifier = estimator.Estimator(model_fn=model_fn)
classifier.train( classifier.train(
input_fn=test_utils.make_input_fn(train_features, train_labels, True), input_fn=test_utils.make_input_fn(train_features, train_labels, True),

View file

@ -14,16 +14,16 @@
"""Multiclass head for Estimator that allow integration with TF Privacy.""" """Multiclass head for Estimator that allow integration with TF Privacy."""
import tensorflow as tf import tensorflow as tf
from tensorflow import estimator as tf_estimator
from tensorflow.python.keras.utils import losses_utils # pylint: disable=g-direct-tensorflow-import from tensorflow.python.keras.utils import losses_utils # pylint: disable=g-direct-tensorflow-import
from tensorflow_estimator.python.estimator import model_fn from tensorflow_estimator.python.estimator import model_fn
from tensorflow_estimator.python.estimator.canned import prediction_keys from tensorflow_estimator.python.estimator.canned import prediction_keys
from tensorflow_estimator.python.estimator.export import export_output from tensorflow_estimator.python.estimator.export import export_output
from tensorflow_estimator.python.estimator.head import base_head from tensorflow_estimator.python.estimator.head import base_head
from tensorflow_estimator.python.estimator.head import multi_label_head
from tensorflow_estimator.python.estimator.mode_keys import ModeKeys from tensorflow_estimator.python.estimator.mode_keys import ModeKeys
class DPMultiLabelHead(tf_estimator.MultiLabelHead): class DPMultiLabelHead(multi_label_head.MultiLabelHead):
"""Creates a TF Privacy-enabled version of MultiLabelHead.""" """Creates a TF Privacy-enabled version of MultiLabelHead."""
def __init__(self, def __init__(self,

View file

@ -14,10 +14,10 @@
import numpy as np import numpy as np
import tensorflow as tf import tensorflow as tf
from tensorflow import estimator as tf_estimator
from tensorflow_privacy.privacy.estimators import multi_label_head from tensorflow_privacy.privacy.estimators import multi_label_head
from tensorflow_privacy.privacy.estimators import test_utils from tensorflow_privacy.privacy.estimators import test_utils
from tensorflow_privacy.privacy.optimizers.dp_optimizer_keras import DPKerasSGDOptimizer from tensorflow_privacy.privacy.optimizers.dp_optimizer_keras import DPKerasSGDOptimizer
from tensorflow_estimator.python.estimator import estimator
class DPMultiLabelHeadTest(tf.test.TestCase): class DPMultiLabelHeadTest(tf.test.TestCase):
@ -64,7 +64,7 @@ class DPMultiLabelHeadTest(tf.test.TestCase):
noise_multiplier=0.0, noise_multiplier=0.0,
num_microbatches=2) num_microbatches=2)
model_fn = test_utils.make_model_fn(head, optimizer, feature_columns) model_fn = test_utils.make_model_fn(head, optimizer, feature_columns)
classifier = tf_estimator.Estimator(model_fn=model_fn) classifier = estimator.Estimator(model_fn=model_fn)
classifier.train( classifier.train(
input_fn=test_utils.make_input_fn(train_features, train_labels, True), input_fn=test_utils.make_input_fn(train_features, train_labels, True),

View file

@ -23,10 +23,7 @@ py_library(
"dnn.py", "dnn.py",
], ],
srcs_version = "PY3", srcs_version = "PY3",
deps = [ deps = [":head"],
":head",
"//third_party/py/tensorflow:tensorflow_estimator",
],
) )
py_test( py_test(
@ -39,7 +36,6 @@ py_test(
":head", ":head",
"//tensorflow_privacy/privacy/estimators:test_utils", "//tensorflow_privacy/privacy/estimators:test_utils",
"//tensorflow_privacy/privacy/optimizers:dp_optimizer", "//tensorflow_privacy/privacy/optimizers:dp_optimizer",
"//third_party/py/tensorflow:tensorflow_estimator",
], ],
) )

View file

@ -16,13 +16,12 @@
import tensorflow as tf import tensorflow as tf
from tensorflow import estimator as tf_estimator
from tensorflow_privacy.privacy.estimators.v1 import head as head_lib from tensorflow_privacy.privacy.estimators.v1 import head as head_lib
from tensorflow_estimator.python.estimator import estimator from tensorflow_estimator.python.estimator import estimator
from tensorflow_estimator.python.estimator.canned import dnn from tensorflow_estimator.python.estimator.canned import dnn
class DNNClassifier(tf_estimator.Estimator): class DNNClassifier(estimator.Estimator):
"""DP version of `tf.compat.v1.estimator.DNNClassifier`.""" """DP version of `tf.compat.v1.estimator.DNNClassifier`."""
def __init__( def __init__(

View file

@ -15,10 +15,10 @@
from absl.testing import parameterized from absl.testing import parameterized
import tensorflow as tf import tensorflow as tf
from tensorflow import estimator as tf_estimator
from tensorflow_privacy.privacy.estimators import test_utils from tensorflow_privacy.privacy.estimators import test_utils
from tensorflow_privacy.privacy.estimators.v1 import head as head_lib from tensorflow_privacy.privacy.estimators.v1 import head as head_lib
from tensorflow_privacy.privacy.optimizers.dp_optimizer import DPGradientDescentGaussianOptimizer from tensorflow_privacy.privacy.optimizers.dp_optimizer import DPGradientDescentGaussianOptimizer
from tensorflow_estimator.python.estimator import estimator
def make_model_fn(head, optimizer, feature_columns): def make_model_fn(head, optimizer, feature_columns):
@ -70,7 +70,7 @@ class DPHeadTest(tf.test.TestCase, parameterized.TestCase):
noise_multiplier=0.0, noise_multiplier=0.0,
num_microbatches=2) num_microbatches=2)
model_fn = make_model_fn(head, optimizer, feature_columns) model_fn = make_model_fn(head, optimizer, feature_columns)
classifier = tf_estimator.Estimator(model_fn=model_fn) classifier = estimator.Estimator(model_fn=model_fn)
classifier.train( classifier.train(
input_fn=test_utils.make_input_fn(train_features, train_labels, True), input_fn=test_utils.make_input_fn(train_features, train_labels, True),