CPU, TOKEN, env variables

#4
by meg HF staff - opened
No description provided.
  • Add CPU/float32 support
  • Fix TOKEN auth
  • Don't redefine env variable (import it)
meg changed pull request status to open

Summary of main changes:

  • Support for 32-bit/CPU: There’s an envs.py variable for DEVICE, where users can say they are using a CPU. But if they do, the code crashes: the code uses float16, which is GPU-specific. Added a simple code thread-through that uses float32 if there’s a CPU chosen as DEVICE.

  • Un-hard-coding environment variables: One file where environment variables were accidentally hard-coded; so I imported the variables from the envs.py instead.

  • Authorization issues: The user access token (TOKEN) must be passed in a few places, such as the snapshot_download functions, in order to play nicely with content that is private.

cc @clefourrier

Demo leaderboard with an integrated backend org

LGTM, thanks a lot for taking the time!

clefourrier changed pull request status to merged

Sign up or log in to comment