Skip to content

Update README.md to explicitly state model linking and download policy #444

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

Merged
merged 1 commit into from
Mar 24, 2023

Conversation

gjmulder
Copy link
Collaborator

Added explicit bolded instructions clarifying that people need to request access to models from Facebook and never through through this repo.

Added explicit **bolded** instructions clarifying that people need to request access to models from Facebook and never through through this repo.
@gjmulder gjmulder changed the title Update README.md Update README.md to explicitly state model linking and download policy Mar 23, 2023
@gjmulder gjmulder added documentation Improvements or additions to documentation model Model specific labels Mar 23, 2023
Copy link
Collaborator

@j-f1 j-f1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice improvement — I think it might be better to call this out at the top so people don’t miss it but this is still a clear improvement.

* The following command will verify if you have all possible latest files in your self-installed `./models` subdirectory:
- **Under no circumstances share IPFS, magnet links, or any other links to model downloads anywhere in this respository, including in issues, discussions or pull requests. They will be immediately deleted.**
- The LLaMA models are officially distributed by Facebook and will **never** be provided through this repository.
- Refer to [Facebook's LLaMA repository](https://github.com/facebookresearch/llama/pull/73/files) if you need to request access to the model data.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is borderline IMO

@gjmulder gjmulder merged commit f4f5362 into master Mar 24, 2023
@j-f1 j-f1 deleted the model-linking-policy-doc branch March 24, 2023 15:28
Deadsg pushed a commit to Deadsg/llama.cpp that referenced this pull request Dec 19, 2023
….99.1

Bump fastapi from 0.98.0 to 0.99.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation model Model specific
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants