Skip to content

fix: correct PYTHONBUFFERED typo to PYTHONUNBUFFERED#1

Open
koriyoshi2041 wants to merge 1 commit intoGen-Verse:mainfrom
koriyoshi2041:fix/pythonunbuffered-typo
Open

fix: correct PYTHONBUFFERED typo to PYTHONUNBUFFERED#1
koriyoshi2041 wants to merge 1 commit intoGen-Verse:mainfrom
koriyoshi2041:fix/pythonunbuffered-typo

Conversation

@koriyoshi2041
Copy link
Copy Markdown

Summary

  • Fix environment variable name PYTHONBUFFEREDPYTHONUNBUFFERED in both launch scripts
  • Fix value from 16 to 1 (the canonical value for enabling unbuffered mode)

PYTHONBUFFERED is not a recognized Python environment variable — Python silently ignores it. The correct variable is PYTHONUNBUFFERED, which disables stdout/stderr buffering when set to any non-empty string.

Without this fix, Python output remains fully buffered when piped (e.g., via ray job submit), causing delayed or lost log output during long-running RL training jobs.

Files changed

  • openclaw-rl/run_qwen3_4b_openclaw_rl.sh
  • openclaw-opd/run_qwen3_4b_openclaw_opd.sh

Test plan

  • Verified PYTHONUNBUFFERED is the correct env var per Python docs
  • Verified 1 is the standard value (boolean flag, any non-empty string enables it)
  • Confirmed no other in-scope files have this typo

The environment variable `PYTHONBUFFERED` is not recognized by Python.
The correct variable name is `PYTHONUNBUFFERED`, which disables output
buffering for stdout/stderr. Additionally, the canonical value for
enabling unbuffered mode is `1`, not `16`.

Without this fix, Python output buffering remains at its default,
which can cause delayed or missing log output — particularly
problematic when debugging long-running RL training jobs.
JasonOA888 pushed a commit to JasonOA888/OpenClaw-RL that referenced this pull request Mar 12, 2026
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