From 0832e8a578f594ac27cc021e836be7143afa2084 Mon Sep 17 00:00:00 2001 From: Alex Strick van Linschoten Date: Fri, 28 Nov 2025 14:04:09 +0100 Subject: [PATCH] Fix weave import side-effects when enable_weave=False Previously, the wandb experiment tracker imported weave and called weave.init(..., settings={"disabled": True}) even when enable_weave was False. With weave >= 0.52.17, simply importing weave triggers wandb_init_hook and remote project creation, causing PERMISSION_ERROR for users not logged in. The fix removes the unnecessary else branch entirely. When enable_weave=False, we now skip all weave-related code, preventing any import side-effects. --- .../experiment_trackers/wandb_experiment_tracker.py | 9 --------- 1 file changed, 9 deletions(-) diff --git a/src/zenml/integrations/wandb/experiment_trackers/wandb_experiment_tracker.py b/src/zenml/integrations/wandb/experiment_trackers/wandb_experiment_tracker.py index 4ec35d85895..d718b3f8c5e 100644 --- a/src/zenml/integrations/wandb/experiment_trackers/wandb_experiment_tracker.py +++ b/src/zenml/integrations/wandb/experiment_trackers/wandb_experiment_tracker.py @@ -180,12 +180,3 @@ def _initialize_wandb( "Weave enabled but no project_name specified. " "Skipping weave initialization." ) - else: - import weave - - if self.config.project_name: - logger.info("Disabling weave") - weave.init( - project_name=self.config.project_name, - settings={"disabled": True}, - )