Skip to content

Commit 0832e8a

Browse files
committed
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.
1 parent b5ee953 commit 0832e8a

File tree

1 file changed

+0
-9
lines changed

1 file changed

+0
-9
lines changed

src/zenml/integrations/wandb/experiment_trackers/wandb_experiment_tracker.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -180,12 +180,3 @@ def _initialize_wandb(
180180
"Weave enabled but no project_name specified. "
181181
"Skipping weave initialization."
182182
)
183-
else:
184-
import weave
185-
186-
if self.config.project_name:
187-
logger.info("Disabling weave")
188-
weave.init(
189-
project_name=self.config.project_name,
190-
settings={"disabled": True},
191-
)

0 commit comments

Comments
 (0)