Closed PavelCz closed 7 months ago
Thx @PavelCz -- looks great. Any chance you can add to run_dpo.py
too? Trying to keep those scripts in sync.
@natolambert, I added the same changes to run_dpo.py
. I can run that file locally as well now.
LGTM. Should be able to merge once the workflows run. I'll probably handle it! Thanks!
Looking forward to using this project!
Here are some minor changes I made to be able to run
run_rm.py
locally:/output/metric.json
which won't be writable on most machines, causing the script to fail.