From 1c110089d8faf10250a80ed61824b530df6c01ce Mon Sep 17 00:00:00 2001 From: Oleh Prypin Date: Tue, 4 Feb 2025 09:04:14 -0800 Subject: [PATCH] Remove `srcs_version` and `python_version` attributes, as they already default to `"PY3"` PiperOrigin-RevId: 723108926 --- examples/distributed_training_ranking_python/BUILD | 1 - examples/vizier_tuning_python/BUILD | 1 - yggdrasil_decision_forests/cli/monitoring/BUILD | 2 -- yggdrasil_decision_forests/port/python/ydf/BUILD | 1 - .../port/python/ydf/dataset/BUILD | 3 --- yggdrasil_decision_forests/port/python/ydf/deep/BUILD | 7 ------- .../port/python/ydf/learner/BUILD | 11 ----------- .../port/python/ydf/metric/BUILD | 2 -- .../port/python/ydf/model/BUILD | 7 ------- .../port/python/ydf/model/decision_forest_model/BUILD | 1 - .../ydf/model/gradient_boosted_trees_model/BUILD | 1 - .../python/ydf/model/isolation_forest_model/BUILD | 1 - .../port/python/ydf/model/random_forest_model/BUILD | 1 - .../port/python/ydf/model/tree/BUILD | 5 ----- .../port/python/ydf/monitoring/BUILD | 1 - yggdrasil_decision_forests/port/python/ydf/util/BUILD | 1 - .../port/python/ydf/utils/BUILD | 5 ----- 17 files changed, 51 deletions(-) diff --git a/examples/distributed_training_ranking_python/BUILD b/examples/distributed_training_ranking_python/BUILD index 64b2391b..d40e7abf 100644 --- a/examples/distributed_training_ranking_python/BUILD +++ b/examples/distributed_training_ranking_python/BUILD @@ -9,7 +9,6 @@ package(licenses = ["notice"]) py_binary( name = "main", srcs = ["main.py"], - python_version = "PY3", deps = [ # absl:app dep, # absl/flags dep, diff --git a/examples/vizier_tuning_python/BUILD b/examples/vizier_tuning_python/BUILD index 64b2391b..d40e7abf 100644 --- a/examples/vizier_tuning_python/BUILD +++ b/examples/vizier_tuning_python/BUILD @@ -9,7 +9,6 @@ package(licenses = ["notice"]) py_binary( name = "main", srcs = ["main.py"], - python_version = "PY3", deps = [ # absl:app dep, # absl/flags dep, diff --git a/yggdrasil_decision_forests/cli/monitoring/BUILD b/yggdrasil_decision_forests/cli/monitoring/BUILD index 61f7ef7c..68d49739 100644 --- a/yggdrasil_decision_forests/cli/monitoring/BUILD +++ b/yggdrasil_decision_forests/cli/monitoring/BUILD @@ -47,8 +47,6 @@ cc_binary_ydf( py_binary( name = "compare_benchmark", srcs = ["compare_benchmark.py"], - python_version = "PY3", - srcs_version = "PY3", deps = [ # absl:app dep, # absl/flags dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/BUILD b/yggdrasil_decision_forests/port/python/ydf/BUILD index 430f8d60..79913dc0 100644 --- a/yggdrasil_decision_forests/port/python/ydf/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/BUILD @@ -104,7 +104,6 @@ py_test( name = "api_test", srcs = ["api_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", shard_count = 5, deps = [ ":api", diff --git a/yggdrasil_decision_forests/port/python/ydf/dataset/BUILD b/yggdrasil_decision_forests/port/python/ydf/dataset/BUILD index 26351054..17366337 100644 --- a/yggdrasil_decision_forests/port/python/ydf/dataset/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/dataset/BUILD @@ -71,7 +71,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", deps = [ ":dataset", ":dataspec", @@ -90,7 +89,6 @@ py_test( name = "dataset_with_tf_test", srcs = ["dataset_with_tf_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", deps = [ ":dataset", # absl/testing:absltest dep, @@ -103,7 +101,6 @@ py_test( py_test( name = "dataspec_test", srcs = ["dataspec_test.py"], - python_version = "PY3", deps = [ ":dataspec", # absl/testing:absltest dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/deep/BUILD b/yggdrasil_decision_forests/port/python/ydf/deep/BUILD index deba3b0f..9738e064 100644 --- a/yggdrasil_decision_forests/port/python/ydf/deep/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/deep/BUILD @@ -154,7 +154,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", shard_count = 4, deps = [ ":mlp", @@ -178,7 +177,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", deps = [ ":dataset", # absl/testing:absltest dep, @@ -191,7 +189,6 @@ py_test( py_test( name = "preprocessor_test", srcs = ["preprocessor_test.py"], - python_version = "PY3", deps = [ ":dataset", ":layer", @@ -210,7 +207,6 @@ py_test( name = "hyperparameter_test", srcs = ["hyperparameter_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", deps = [ ":hyperparameter", # absl/testing:absltest dep, @@ -221,7 +217,6 @@ py_test( py_test( name = "layer_test", srcs = ["layer_test.py"], - python_version = "PY3", deps = [ ":layer", # absl/testing:absltest dep, @@ -233,7 +228,6 @@ py_test( py_test( name = "metric_test", srcs = ["metric_test.py"], - python_version = "PY3", deps = [ ":metric", # absl/testing:absltest dep, @@ -249,7 +243,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", shard_count = 4, deps = [ ":layer", diff --git a/yggdrasil_decision_forests/port/python/ydf/learner/BUILD b/yggdrasil_decision_forests/port/python/ydf/learner/BUILD index ffcb72f4..152a1f9e 100644 --- a/yggdrasil_decision_forests/port/python/ydf/learner/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/learner/BUILD @@ -15,7 +15,6 @@ package( py_binary( name = "worker_main", srcs = ["worker_main.py"], - python_version = "PY3", visibility = ["//visibility:public"], deps = [ # absl:app dep, @@ -208,7 +207,6 @@ py_library( py_library( name = "tuner", srcs = ["tuner.py"], - srcs_version = "PY3", deps = [ "@ydf_cc//yggdrasil_decision_forests/learner:abstract_learner_py_proto", "@ydf_cc//yggdrasil_decision_forests/learner/hyperparameters_optimizer:hyperparameters_optimizer_py_proto", @@ -220,7 +218,6 @@ py_library( py_library( name = "worker", srcs = ["worker.py"], - srcs_version = "PY3", deps = [ "//ydf/cc:ydf", ], @@ -253,7 +250,6 @@ py_test( size = "large", srcs = ["learner_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", shard_count = 10, tags = [ ], @@ -291,7 +287,6 @@ py_test( timeout = "long", srcs = ["custom_loss_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", deps = [ ":custom_loss_py", ":generic_learner", @@ -312,7 +307,6 @@ py_test( name = "distributed_learner_test", srcs = ["distributed_learner_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", shard_count = 10, deps = [ ":generic_learner", @@ -332,7 +326,6 @@ py_test( py_test( name = "learner_with_tf_test", srcs = ["learner_with_tf_test.py"], - python_version = "PY3", deps = [ ":generic_learner", ":specialized_learners", @@ -346,7 +339,6 @@ py_test( py_test( name = "hyperparameters_test", srcs = ["hyperparameters_test.py"], - python_version = "PY3", deps = [ ":hyperparameters", # absl/testing:absltest dep, @@ -359,7 +351,6 @@ py_test( name = "learner_with_xarray_test", srcs = ["learner_with_xarray_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", deps = [ ":specialized_learners", # absl/testing:absltest dep, @@ -372,7 +363,6 @@ py_test( py_test( name = "tuner_test", srcs = ["tuner_test.py"], - python_version = "PY3", tags = [ ], deps = [ @@ -391,7 +381,6 @@ py_test( name = "feature_selector_test", srcs = ["feature_selector_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", shard_count = 5, deps = [ ":feature_selector", diff --git a/yggdrasil_decision_forests/port/python/ydf/metric/BUILD b/yggdrasil_decision_forests/port/python/ydf/metric/BUILD index 6a16c833..39df75a3 100644 --- a/yggdrasil_decision_forests/port/python/ydf/metric/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/metric/BUILD @@ -81,7 +81,6 @@ py_test( name = "metric_test", srcs = ["metric_test.py"], data = ["//test_data"], - python_version = "PY3", deps = [ ":metric", # absl/testing:absltest dep, @@ -96,7 +95,6 @@ py_test( name = "evaluate_test", srcs = ["evaluate_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", deps = [ ":evaluate", # absl/testing:absltest dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/model/BUILD b/yggdrasil_decision_forests/port/python/ydf/model/BUILD index 46cdd2f4..d6eb6194 100644 --- a/yggdrasil_decision_forests/port/python/ydf/model/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/model/BUILD @@ -245,7 +245,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", shard_count = 4, deps = [ ":analysis", @@ -273,7 +272,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", shard_count = 5, # TODO: This test fails in OSS due to a package name clash on yggdrasil_decision_forests. tags = ["manual"], @@ -330,7 +328,6 @@ genrule( py_binary( name = "export_cc_generator", srcs = ["export_cc_generator.py"], - python_version = "PY3", deps = [ # absl:app dep, # absl/flags dep, @@ -342,7 +339,6 @@ py_binary( py_test( name = "optimizer_logs_test", srcs = ["optimizer_logs_test.py"], - python_version = "PY3", deps = [ ":optimizer_logs", # absl/testing:absltest dep, @@ -360,7 +356,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", deps = [ ":export_jax", ":generic_model", @@ -385,7 +380,6 @@ py_test( py_test( name = "sklearn_model_test", srcs = ["sklearn_model_test.py"], - python_version = "PY3", deps = [ ":export_sklearn", # absl/testing:absltest dep, @@ -400,7 +394,6 @@ py_test( name = "export_docker_test", srcs = ["export_docker_test.py"], data = ["@ydf_cc//yggdrasil_decision_forests/test_data"], - python_version = "PY3", deps = [ ":export_docker", ":model_lib", diff --git a/yggdrasil_decision_forests/port/python/ydf/model/decision_forest_model/BUILD b/yggdrasil_decision_forests/port/python/ydf/model/decision_forest_model/BUILD index aef790f4..7ef6a9b8 100644 --- a/yggdrasil_decision_forests/port/python/ydf/model/decision_forest_model/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/model/decision_forest_model/BUILD @@ -52,7 +52,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", deps = [ ":decision_forest_model", # absl/testing:absltest dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/model/gradient_boosted_trees_model/BUILD b/yggdrasil_decision_forests/port/python/ydf/model/gradient_boosted_trees_model/BUILD index 3d07e6f6..9a960c7a 100644 --- a/yggdrasil_decision_forests/port/python/ydf/model/gradient_boosted_trees_model/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/model/gradient_boosted_trees_model/BUILD @@ -48,7 +48,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", deps = [ ":gradient_boosted_trees_model", # absl/logging dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/model/isolation_forest_model/BUILD b/yggdrasil_decision_forests/port/python/ydf/model/isolation_forest_model/BUILD index d2bb104d..5481cd3b 100644 --- a/yggdrasil_decision_forests/port/python/ydf/model/isolation_forest_model/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/model/isolation_forest_model/BUILD @@ -42,7 +42,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", deps = [ # absl/testing:absltest dep, # numpy dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/model/random_forest_model/BUILD b/yggdrasil_decision_forests/port/python/ydf/model/random_forest_model/BUILD index e43292cc..ef372402 100644 --- a/yggdrasil_decision_forests/port/python/ydf/model/random_forest_model/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/model/random_forest_model/BUILD @@ -46,7 +46,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", deps = [ # absl/testing:absltest dep, # numpy dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/model/tree/BUILD b/yggdrasil_decision_forests/port/python/ydf/model/tree/BUILD index 955830c6..118971e3 100644 --- a/yggdrasil_decision_forests/port/python/ydf/model/tree/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/model/tree/BUILD @@ -88,7 +88,6 @@ py_test( data = [ "//test_data", ], - python_version = "PY3", deps = [ ":condition", ":node", @@ -104,7 +103,6 @@ py_test( py_test( name = "value_test", srcs = ["value_test.py"], - python_version = "PY3", deps = [ ":value", # absl/testing:absltest dep, @@ -117,7 +115,6 @@ py_test( py_test( name = "node_test", srcs = ["node_test.py"], - python_version = "PY3", deps = [ ":condition", ":node", @@ -129,7 +126,6 @@ py_test( py_test( name = "condition_test", srcs = ["condition_test.py"], - python_version = "PY3", deps = [ ":condition", # absl/testing:absltest dep, @@ -144,7 +140,6 @@ py_test( name = "plot_test", srcs = ["plot_test.py"], data = ["assets"], - python_version = "PY3", deps = [ ":plot", # absl/testing:absltest dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/monitoring/BUILD b/yggdrasil_decision_forests/port/python/ydf/monitoring/BUILD index 28f5fc68..852e2008 100644 --- a/yggdrasil_decision_forests/port/python/ydf/monitoring/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/monitoring/BUILD @@ -81,7 +81,6 @@ py_library( py_test( name = "benchmark_inference_speed_test", srcs = ["benchmark_inference_speed_test.py"], - python_version = "PY3", deps = [ ":benchmark_inference_speed", # absl/testing:absltest dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/util/BUILD b/yggdrasil_decision_forests/port/python/ydf/util/BUILD index 6645d9ff..fbd58d07 100644 --- a/yggdrasil_decision_forests/port/python/ydf/util/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/util/BUILD @@ -48,7 +48,6 @@ py_test( "//test_data", "@ydf_cc//yggdrasil_decision_forests/test_data", ], - python_version = "PY3", deps = [ ":tf_example", # absl/logging dep, diff --git a/yggdrasil_decision_forests/port/python/ydf/utils/BUILD b/yggdrasil_decision_forests/port/python/ydf/utils/BUILD index 1279b8bb..e7bfe504 100644 --- a/yggdrasil_decision_forests/port/python/ydf/utils/BUILD +++ b/yggdrasil_decision_forests/port/python/ydf/utils/BUILD @@ -108,7 +108,6 @@ py_library( py_test( name = "func_helpers_test", srcs = ["func_helpers_test.py"], - python_version = "PY3", deps = [ ":func_helpers", # absl/testing:absltest dep, @@ -118,7 +117,6 @@ py_test( py_test( name = "html_test", srcs = ["html_test.py"], - python_version = "PY3", deps = [ ":html", # absl/testing:absltest dep, @@ -128,7 +126,6 @@ py_test( py_test( name = "paths_test", srcs = ["paths_test.py"], - python_version = "PY3", deps = [ ":paths", # absl/testing:absltest dep, @@ -138,7 +135,6 @@ py_test( py_test( name = "string_lib_test", srcs = ["string_lib_test.py"], - python_version = "PY3", deps = [ ":string_lib", # absl/testing:absltest dep, @@ -148,7 +144,6 @@ py_test( py_test( name = "test_utils_test", srcs = ["test_utils_test.py"], - python_version = "PY3", deps = [ ":test_utils", # absl/testing:absltest dep,