Skip to content

fix: cast Path to str when spawning tail subprocess#1824

Open
enlightened88 wants to merge 1 commit intoPrimeIntellect-ai:mainfrom
enlightened88:patch-1
Open

fix: cast Path to str when spawning tail subprocess#1824
enlightened88 wants to merge 1 commit intoPrimeIntellect-ai:mainfrom
enlightened88:patch-1

Conversation

@enlightened88
Copy link

@enlightened88 enlightened88 commented Feb 18, 2026

Ensure subprocess receives string path when spawning tail to avoid potential cross-platform Path handling issues


Note

Low Risk
Single-argument type fix when launching a non-critical tail subprocess; minimal behavioral impact beyond improved compatibility.

Overview
Fixes RL log streaming startup by converting the Path for trainer.stdout to a plain string when spawning tail -F via Popen, avoiding Path/subprocess argument handling issues on some platforms.

Written by Cursor Bugbot for commit 8e9bd76. This will update automatically on new commits. Configure here.

Signed-off-by: Андрій Шовкошитний <198119344+enlightened88@users.noreply.github.com>
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