Skip to content

Commit 150bd41

Browse files
authored
ensure trackers are used only on master process (#578)
Signed-off-by: Dushyant Behl <dushyantbehl@in.ibm.com>
1 parent a8dd542 commit 150bd41

File tree

3 files changed

+15
-0
lines changed

3 files changed

+15
-0
lines changed

tuning/trackers/aimstack_tracker.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ class RunIDExporterAimCallback(AimCallback):
2929
as soon as it is created, which is during on_init_end.
3030
"""
3131

32+
_use_aimstack = True
3233
# path where we export run hash generated by Aim
3334
# This is used to link back to the expriments from outside aimstack
3435
run_id_export_path = None
@@ -62,11 +63,15 @@ def on_init_end(self, args, state, control, **kwargs):
6263
"""
6364
super().on_init_end(args, state, control, **kwargs)
6465

66+
if not self._use_aimstack or not state.is_world_process_zero:
67+
return
68+
6569
if not self._run:
6670
self.logger.warning(
6771
"Aim tracker was requested but did not get initialized;"
6872
+ " Please check the config"
6973
)
74+
self._use_aimstack = False
7075
return
7176

7277
if self.tracker:

tuning/trackers/clearml_tracker.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ class RunURIExporterClearMLCallback(ClearMLCallback):
3333
as soon as it is created, which is on setup().
3434
"""
3535

36+
_use_clearml = True
3637
run_uri_export_path: str = None
3738
logger = None
3839
tracker: Tracker = None
@@ -80,11 +81,15 @@ def setup(self, args, state, model, processing_class, **kwargs):
8081
self._clearml = None
8182
return
8283

84+
if not self._use_clearml or not state.is_world_process_zero:
85+
return
86+
8387
if not self._initialized or self._clearml_task is None:
8488
self.logger.warning(
8589
"ClearMLtracker was requested but did not get initialized;"
8690
+ " Please check the config"
8791
)
92+
self._use_clearml = False
8893
return
8994

9095
task = self._clearml.Task.current_task()

tuning/trackers/mlflow_tracker.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ class RunURIExporterMlflowCallback(MLflowCallback):
3030
as soon as it is created, which is on setup().
3131
"""
3232

33+
_use_mlflow = True
3334
run_uri_export_path: str = None
3435
tracker: Tracker
3536
client = None
@@ -60,11 +61,15 @@ def setup(self, args, state, model):
6061
"""
6162
super().setup(args, state, model)
6263

64+
if not self._use_mlflow or not state.is_world_process_zero:
65+
return
66+
6367
if not self._initialized:
6468
self.logger.warning(
6569
"mlflow tracker was requested but did not get initialized;"
6670
+ " Please check the config"
6771
)
72+
self._use_mlflow = False
6873
return
6974

7075
self.client = self._ml_flow

0 commit comments

Comments
 (0)