Skip to content

Commit 92cc542

Browse files
Pathways-on-Cloud Teamcopybara-github
authored andcommitted
Remove support for legacy initialize
PiperOrigin-RevId: 748409506
1 parent 0e6f105 commit 92cc542

2 files changed

Lines changed: 10 additions & 16 deletions

File tree

pathwaysutils/__init__.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -83,12 +83,12 @@ def initialize() -> None:
8383
# debug log is not triggered for customers who are following our instructions
8484
# and using the new initialize() function only once but have already had the
8585
# legacy initialization triggered.
86-
if _initialization_count > 2:
87-
_logger.debug("Already initialized. Ignoring duplicate call.")
88-
8986
if _initialization_count > 1:
87+
_logger.debug("Already initialized. Ignoring duplicate call.")
9088
return
9189

90+
_logger.debug("Starting initialize.")
91+
9292
if is_pathways_backend_used():
9393
_logger.debug("Detected Pathways-on-Cloud backend. Applying changes.")
9494
proxy_backend.register_backend_factory()
@@ -112,6 +112,3 @@ def initialize() -> None:
112112
_logger.debug(
113113
"Did not detect Pathways-on-Cloud backend. No changes applied."
114114
)
115-
116-
117-
initialize()

pathwaysutils/test/pathwaysutils_test.py

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -43,24 +43,21 @@ def test_first_initialize(self):
4343
with self.assertLogs(pathwaysutils._logger, level="DEBUG") as logs:
4444
pathwaysutils.initialize()
4545

46-
self.assertLen(logs.output, 1)
46+
self.assertLen(logs.output, 2)
4747
self.assertIn(
48-
"Detected Pathways-on-Cloud backend. Applying changes.", logs.output[0]
48+
"Starting initialize.", logs.output[0]
49+
)
50+
self.assertIn(
51+
"Detected Pathways-on-Cloud backend. Applying changes.", logs.output[1]
4952
)
50-
51-
def test_second_initialize(self):
52-
jax.config.update("jax_platforms", "proxy")
53-
pathwaysutils._initialization_count = 1
54-
55-
with self.assertNoLogs(pathwaysutils._logger, level="DEBUG"):
56-
pathwaysutils.initialize()
5753

5854
@parameterized.named_parameters(
55+
("initialization_count 1", 1),
5956
("initialization_count 2", 2),
6057
("initialization_count 5", 5),
6158
("initialization_count 1000", 1000),
6259
)
63-
def test_initialize_more_than_twice(self, initialization_count):
60+
def test_initialize_more_than_once(self, initialization_count):
6461
pathwaysutils._initialization_count = initialization_count
6562

6663
with self.assertLogs(pathwaysutils._logger, level="DEBUG") as logs:

0 commit comments

Comments
 (0)