Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Issue with CACHE_DIR Default Value in Accelerate Pipeline #460

Open
ryan-minato opened this issue Dec 19, 2024 · 0 comments
Open

[BUG] Issue with CACHE_DIR Default Value in Accelerate Pipeline #460

ryan-minato opened this issue Dec 19, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@ryan-minato
Copy link
Contributor

Describe the bug

The default value for CACHE_DIR in the pipeline creates several issues. On many systems, such as Ubuntu distributions, users typically lack permissions for the root directory (/), which prevents the pipeline from running. On Windows, it creates a scratch folder on the C drive, which is not ideal.

CACHE_DIR: str = os.getenv("HF_HOME", "/scratch")

Proposed Solution

Remove the default value for CACHE_DIR and instead return None. This will allow the pipeline to follow the default logic of the transformers library.

CACHE_DIR: str = os.getenv("HF_HOME")

To Reproduce

Run the example provided in the README on any Linux distribution:

HF_HOME= lighteval accelerate \
    "pretrained=gpt2" \
    "leaderboard|truthfulqa:mc|0|0"

Expected behavior

The running should execute without errors.

Version info

The issue was encountered on the main branch installed with pip install -e .[dev].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant