togethercomputer / together-python

The Official Python Client for Together's API
https://pypi.org/project/together/
Apache License 2.0
24 stars 5 forks source link

Add wandb api key length check #64

Closed azahed98 closed 5 months ago

azahed98 commented 9 months ago

Issue # https://linear.app/together-ai/issue/ENG-450/fine-tune-70b-job-failed-due-to-wandb-response-401

Describe your changes

Adds a check to make sure the WandB api key (if provided) is the correct length. This matches the check in the WandB library.

Note that this does not mean the api key is valid. Determining if it is possible to determine this conclusively without starting a new run. It seems WandB does not explicitly support this.

orangetin commented 5 months ago

needs to be rebased