Skip to content

Conversation

robtinn
Copy link
Contributor

@robtinn robtinn commented May 21, 2025

Summary

New cookbook to showcase the RFT capabilities in the OpenAI API.

Motivation

Motivation for this was I was fixing/updating some of the old fine-tuning cookbooks. The biggest challenge with upgrading them is model performance has improved so much that we need new challenging datasets to show substantial finetuning gains (which is where HealthBench is ideal).

RFT is also a new feature with few examples and this cookbook covers it end-to-end including working with evals.

@robtinn robtinn requested a review from tompakeman-oai May 21, 2025 13:12
Copy link
Contributor

@tompakeman-oai tompakeman-oai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Headlines

  • Overall I think it's great but it needs more contextualising with the use case
    • What problem fine tuning solves, why few shot is helpful, etc.
  • Basically add some user focused language in markdown cells that can be read on the cookbook website by someone who doesn't want to run the code.
  • I also think you want to submit the cookbook with the outputs generated so users can see the charts in the final output.

Other changes

  • Minor python optimisations for opening files
  • Fixes for job polling + using responses API format
  • Bug in the matplotlib chart

@tompakeman-oai
Copy link
Contributor

Looks way better. Final thing is that I think it would be awesome to rerun the same evaluation post fine tuning so we can do a direct before and after on the scoring. Apart from that, LGTM

@robtinn robtinn merged commit 97e80fa into main May 23, 2025
1 check passed
@robtinn robtinn deleted the rtinn/rft branch May 23, 2025 15:26
theophile-oai pushed a commit that referenced this pull request May 23, 2025
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.

2 participants