Use `peft` for RLHF about peft HOT 5 CLOSED

I think it sounds like a good idea if you start out with the gpt2 script and then I can do the t5 script. I think that would be a nice way to do it and I can understand much more by following along with your work.
Btw I'm on the huggingface discord if you want to messag me my username is birger

from peft.

@pacman100 @younesbelkada I could work on this if there is any interest. What would be the first steps?

from peft.

Thanks for your interest @BirgerMoell !! That would be definitely helpful
We need to check first the compatbility of peft and trl (i.e. it might not directly work out of the box but we will need to double check), so we will most likely adapt the gpt2 script ourselves just to make sure we fix things that needs to be fixed, then you can help us maybe converting the t5 script? Let me know what do you think!
If you feel you want to take the challenge and start converting the gpt2 script feel free to do it and we'll be happy to review your Pull Request / code changes ! 🔥

from peft.

The PR huggingface/trl#163 should add peft support on trl, we may need to convert the other scripts such as t5 summarization etc as well once this PR gets merged!

from peft.

Hi everyone,
Closing this issue as we now officially support PEFT in TRL , making it possible to apply RLHF + PEFT

from peft.