Thomas Capelle

Results 169 comments of Thomas Capelle

isn't block size inferred from the tokenizer ?

Not right now, but I would gladly review a PR that adds this feature to the MiniTrainer class. You could use Accelerate or just vanilla DDP.

Hey, can you just use ``` import wandb wandb.login() ``` instead?

Two extra small changes: - remove the entity = "saurav" and default to None (so it' uses your default entity after calling `wandb.login`) - Add the image to the peft...

extra detail, use a relative path to the image, so when we merge the path to your branch is not needed

Ping me when ready for final review/merge

> > Why we don't show the classic: > > ```python > > # you can log using the one-liner: > > wandb.log_artifact("file.csv", name="my_artifact", type="data") > > > > #...