cc @qgallouedec for review when you get the time.
Merci!

Thanks for the fix! We could add a modified version of the script as a test in test_sft_trainer.py and remove the script

Thanks for your quick reply @sergiopaniego !
Done in 6465594 :), and the test is passing fine:

Good to merge on my side (but it looks like we still need approval for the workflows to run)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@sergiopaniego do you understand why the tests are failing ? It doesn't look related to my changes ... :x
Otherwise ready to merge on my side (once this test issue is solved).

Hi, thanks for your contribution. I opted for a different approach: when the dataset does not contain images, it can be pre-processed, and there is no point in doing it on the fly.

Hi, thanks for your contribution. I opted for a different approach: when the dataset does not contain images, it can be pre-processed, and there is no point in doing it on the fly.

This works as well! Thanks for the edit @qgallouedec.