-
-
Notifications
You must be signed in to change notification settings - Fork 780
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve documentation and implementation of W&B options #236
Comments
We need to make sure to migrate any old/incorrect configs and check for old env within |
Improved by #361 |
Sorry, I accidentally closed |
Hello, W&B Eng here, I can help with this. Also we should probably not explicitly set the We could probably have something like this: def setup_wandb_env_vars(cfg):
for key in cfg.__dict__:
if key.startswith('wandb_'):
value = cfg.get(key, "")
if value and len(value) > 0:
os.environ[key.upper()] = value
elif key == 'wandb_mode' and value == "offline":
os.environ["WANDB_MODE"] = value
else:
os.environ["WANDB_DISABLED"] = "true"
# Enable wandb if project name is present
if cfg.get('wandb_project', "") and len(cfg.get('wandb_project', "")) > 0:
cfg.use_wandb = True |
wandb_
to the same env vardiscord discussion: https://discord.com/channels/1104757954588196865/1111279858136383509/1121703569029091412
The text was updated successfully, but these errors were encountered: