Closed carmocca closed 3 weeks ago
@rasbt Do you know if this subcommand is still necessary? Or should I close this?
I think that if we go the litgpt finetune REPO_ID
route, then subcommands would probably not work. It would then probably have to be litgpt finetune REPO_ID --method qlora
. If that's correct, you can close this, but please correct me if I'm wrong.
Code is magic and anything is possible, but in general, you are correct.
Note also that to support that kind of notation, a replacement for jsonargparse will be necessary because its argument introspection capabilities are not complex enough to support that notation. I tried in the past unsuccessfully.
qlora = lora with bnb and true precision.
I think bf16-true is fine as a default because bitsandbytes doesnt' support the old cards well (like my laptop)