Skip to content

Limit W&B tag length to comply with 64‑character limit #995

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

iory
Copy link

@iory iory commented Apr 17, 2025

What this does

wandb.init() rejects any tag longer than 64 characters and returns:

400 Bad Request: invalid tag 'dataset:iory0710/apart_lift_a_white_faucet_start_from_same_pose_5hz'. must be between 1 and 64 characters

This pull request adds a safeguard that truncates every tag to ≤ 64 characters before passing it to wandb.init(). The change prevents the wandb.errors.CommError shown above while leaving the rest of the logging logic untouched.

Examples:

Title Label
Fixes #544 (🐛 Bug)

How it was tested

Tested – Verified locally with the previously failing dataset name iory0710/apart_lift_a_white_faucet_start_from_same_pose_5hz.

How to checkout & try? (for the reviewer)

python lerobot/scripts/train.py   --dataset.repo_id=iory0710/apart_lift_a_white_faucet_start_from_same_pose_5hz   --policy.type=diffusion   --output_dir=outputs/train/diffusion_apart_test   --job_name=diffusion_apart_test   --wandb.enable=true   --policy.num_inference_steps=10   --policy.device=cpu   --steps=200000

@imstevenpmwork imstevenpmwork added bug Something isn’t working correctly visualization Issues about visual output, graphs, or data visualization labels Apr 17, 2025
@iory
Copy link
Author

iory commented Apr 18, 2025

Can someone please approve the workflow so we can run it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn’t working correctly visualization Issues about visual output, graphs, or data visualization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Long dataset names break w&b tagging
2 participants