llama icon indicating copy to clipboard operation
llama copied to clipboard

Explicitly state in README.md to use `bash download.sh` instead of download.sh in case user is not using bash.

Open penguinpatroller opened this issue 1 year ago • 5 comments

For zsh users, this script will throw confusing permission denied errors, even when making the script executable. I think it would be good to put in the README.md that they should use bash download.sh instead of ./download.sh, to avoid this error in the future.

penguinpatroller avatar Apr 28 '23 05:04 penguinpatroller

For zsh users, this script will throw confusing permission denied errors, even when making the script executable. I think it would be good to put in the README.md that they should use bash download.sh instead of ./download.sh, to avoid this error in the future.

Dang, I was struggling a lot due to that! Thanks!

Manurocker95 avatar May 25 '23 07:05 Manurocker95

I run bash download.sh but got the following issues: (LLaMa) C:\Users\dukew\models\llama>bash download.sh download.sh: line 3: $'\r': command not found download.sh: line 7: $'\r': command not found ': not a valid identifierare: N_SHARD_DICT download.sh: line 9: $'\r': command not found download.sh: line 10: 7B: value too great for base (error token is "7B") download.sh: line 11: 13B: value too great for base (error token is "13B") download.sh: line 12: 30B: value too great for base (error token is "30B") download.sh: line 13: 65B: value too great for base (error token is "65B") download.sh: line 14: $'\r': command not found Downloading tokenizer : No such file or directoryizer.model : No such file or directoryizer_checklist.chk download.sh: line 18: $'\r': command not found download.sh: line 19: syntax error near unexpected token $'\r'' 'ownload.sh: line 19: `(cd ${TARGET_FOLDER} && md5sum -c tokenizer_checklist.chk)

What's the issue?

dukewillbe185 avatar May 26 '23 12:05 dukewillbe185

Yep. Kept getting zsh: permission denied on MacOS Ventura until I found this, specifying bash fixed it.

paulmatthis avatar Jul 26 '23 13:07 paulmatthis

Hi @penguinpatroller Following this FAQ, https://github.com/facebookresearch/llama/blob/bug-fix-Sep6/Download.md

I have verified the scripts to be working fine on both ubuntu and mac m1 13.5. Please let me know if this helps

agunapal avatar Sep 06 '23 18:09 agunapal

For zsh users, this script will throw confusing permission denied errors, even when making the script executable. I think it would be good to put in the README.md that they should use bash download.sh instead of ./download.sh, to avoid this error in the future.

Thanks dude!Using sh download.sh in a bash shell also incurs the "permission denied" error. Command bash download.sh solves it.

kyrie2to11 avatar Nov 28 '23 02:11 kyrie2to11