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.

Sign up for GitHub

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

Update model_license_other.md #993

Merged
merged 3 commits into from
Oct 6, 2023
Merged

Update model_license_other.md #993

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
bfb0fd2
Update model_license_other.md
thanktua-spp Oct 3, 2023
25f88c0
Merge branch 'main' into patch-2
Wauplin Oct 6, 2023
d03b7c2
Update hacktoberfest_challenges/model_license_other.md
Wauplin Oct 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion hacktoberfest_challenges/model_license_other.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ Here is the list of models with `license:other` and a LICENSE file in the repo.

|status|pr_url |model_id|nb_downloads|nb_likes |suggested_license_name |suggested_license_link |license_url |license_url_2 |license_url_3 |
|------|----------------------------------------------------------------------------|--------|------------|-------------------------|----------------------------------------------------------------------------------------------|----------------------------------------------------------------------------------------------|---------------------------------------------------------------------------------------------------|---------------------------------------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------|
| | |[decapoda-research/llama-7b-hf](https://huggingface.co/decapoda-research/llama-7b-hf)|379224 |1223 |llama-license |https://huggingface.co/decapoda-research/llama-7b-hf/blob/main/LICENSE |[LICENSE](https://huggingface.co/decapoda-research/llama-7b-hf/blob/main/LICENSE) | | |
|Opened|https://huggingface.co/decapoda-research/llama-7b-hf/discussions/125|[decapoda-research/llama-7b-hf](https://huggingface.co/decapoda-research/llama-7b-hf)|379224 |1223 |llama-license |https://huggingface.co/decapoda-research/llama-7b-hf/blob/main/LICENSE |[LICENSE](https://huggingface.co/decapoda-research/llama-7b-hf/blob/main/LICENSE) | | |
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, this is the idea @thanktua-spp! Thanks for opening the PR 🎉
2 remarks though:

  1. It's better to wrap the PR url like this [here](https://huggingface.co/decapoda-research/llama-7b-hf/discussions/125). This doesn't change much but will make the table more readable on GitHub.
  2. The model you reported is not under Llama2 license, but under the 1st Llama version. In that case it's best to keep license:other, add license_name:llama and finally link to the LICENSE in the repo as license_link: https://huggingface.co/decapoda-research/llama-7b-hf/blob/main/LICENSE

Does that work for you? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks so much, @Wauplin this was very helpful feedback. Many thanks.

| | |[huggyllama/llama-7b](https://huggingface.co/huggyllama/llama-7b)|319220 |200 |llama-license |https://huggingface.co/huggyllama/llama-7b/blob/main/LICENSE |[LICENSE](https://huggingface.co/huggyllama/llama-7b/blob/main/LICENSE) | | |
| | |[TheBloke/MythoMax-L2-13B-GPTQ](https://huggingface.co/TheBloke/MythoMax-L2-13B-GPTQ)|183096 |69 |llama-2-community-license |https://ai.meta.com/llama/license/ |[LICENSE.txt](https://huggingface.co/TheBloke/MythoMax-L2-13B-GPTQ/blob/main/LICENSE.txt) | | |
| | |[elinas/chronos-13b-v2](https://huggingface.co/elinas/chronos-13b-v2)|35941 |16 |llama-2-community-license |https://ai.meta.com/llama/license/ |[LICENSE.txt](https://huggingface.co/elinas/chronos-13b-v2/blob/main/LICENSE.txt) | | |
Expand Down