Repository navigation
Fix/1912 role play preamble - #3309
Joel (aspire488) wants to merge 7 commits into
Conversation
|
Hi Roman, the PR is open and the CLA check has passed. However, the CI workflows are showing "action_required" without executing any jobs. Could you or another maintainer approve the workflow runs so validation can proceed? The regression tests and 50-run before/after comparison are still outstanding. I'll update the PR with the results once validation is complete. Thanks! |
|
Thanks Joel (@aspire488)! Did you try it out as well? Any stats on whether it helped? |
|
Thanks for putting this together, Joel (@aspire488), and sorry for the unnecessary work. I wasn't aware that the July 11 role-play refactor in #2160 had already addressed the original issue when I encouraged work on #1912. That refactor removed the old I verified this with GPT-4o-unsafe and a harmless origami objective: the original June-era prompts produced preambles in 6 of 9 completed generations; one additional historical generation was truncated and excluded. The existing implementation before this PR produced 0 preambles in 12 generations, and this PR also produced 0 in 12, with matched settings within each comparison. The sample is small, but it reproduces the old failure and supports that the July replacement already addressed it. This PR adds further prompt guidance, but we haven't demonstrated an additional improvement on the current implementation. I'm closing this PR without merging and resolving #1912 as already addressed by the July refactor. Thank you again for the contribution. |
Description
Tests and Documentation
Summary
Fixes #1912 by strengthening role-play prompt instructions to prevent conversational preambles and ensure generated messages begin directly with the in-character content.
Changes
role_play_next_message.yamlwith the same requirement.Validation
Testing
Closes #1912