CLI utils class cases seem to be incorrect

[busycalibrating]

The cases for the DPOScriptArguments and SFTScriptArguments (as seen here) have the correctly capitalized DPO and SFT names, but in the downloaded codebase, it seems to be in camel-case, i.e.

@dataclass
class SftScriptArguments:
    ...

@dataclass
class DpoScriptArguments:
    ...

I'm currently running version 0.8.6, installed from pip, but you can also see this in the downloaded zip release of v0.8.6.

Does anybody else see this or is this something just affecting me?

[younesbelkada]

Hi @busycalibrating
Thanks for the issue, when searching into the codebase, only the capitalized DPO and SFT names are present:

https://github.com/search?q=repo%3Ahuggingface%2Ftrl+SftScriptArguments&type=code

https://github.com/search?q=repo%3Ahuggingface%2Ftrl+DpoScriptArguments&type=code

[busycalibrating]

Can you check that this is true when you download the zip release I linked?

[busycalibrating]

Bumping this as I'm still running into this. For context, here's what the file looks like when I download the release I linked above: