Skip to content

fix(32b): use main branch for workdir instead of feat/vl-multimodal-support#198

Merged
dzorlu merged 1 commit intomainfrom
fix/qwen3-32b-workdir-ref
Feb 15, 2026
Merged

fix(32b): use main branch for workdir instead of feat/vl-multimodal-support#198
dzorlu merged 1 commit intomainfrom
fix/qwen3-32b-workdir-ref

Conversation

@dzorlu
Copy link
Collaborator

@dzorlu dzorlu commented Feb 15, 2026

Summary

  • Fix workdir ref in openenv-fleet-grpo-qwen3-32b.yaml to use main instead of feat/vl-multimodal-support
  • The old branch had an outdated prepare_dataset.py that doesn't support comma-separated env_keys

Problem

When running the 32B training workflow with ENV_KEYS=reddit,zillow,forums-homes,..., the job fails with:

FileNotFoundError: Unable to find '/root/data/fleet/tool_use/train.parquet'

Root cause: The feat/vl-multimodal-support branch has the OLD prepare_dataset.py that does exact string matching:

# OLD (feat/vl-multimodal-support)
if env_filter:
    tasks = [t for t in tasks if t.get("env_key") == env_filter]  # exact match

While main has the FIXED version that supports comma-separated values:

# NEW (main)
if env_filter:
    env_list = [e.strip() for e in env_filter.split(",") if e.strip()]
    tasks = [t for t in tasks if t.get("env_key") in env_list]  # list membership

Test plan

  • Re-run the training workflow with the same ENV_KEYS and verify dataset is created correctly

🤖 Generated with Claude Code

…upport

The feat/vl-multimodal-support branch has an outdated prepare_dataset.py
that doesn't support comma-separated env_keys in --env-filter. This caused
training to fail with FileNotFoundError when ENV_KEYS contained multiple
environments (e.g., "reddit,zillow,forums-homes").

The main branch has the fixed prepare_dataset.py that properly splits
comma-separated values.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@dzorlu dzorlu merged commit 6375011 into main Feb 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant