-
Notifications
You must be signed in to change notification settings - Fork 731
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
Suggest pinned commits in config URLs #1145
Labels
enhancement
New feature or request
Comments
I see two additional options:
|
Probably 2 would be better here so users could still use full urls to main if they wanted to deliberately. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Our tutorials have suggestions like
But this has the drawback that it will stop working if:
main
and the user did not upgradeSo we should suggest a specific commit:
https://raw.githubusercontent.com/Lightning-AI/litgpt/396bae5/config_hub/finetune/llama-2-7b/lora.yaml
The disadvantage is that we will need to remember to update this commit. Maybe we could automate this with a GitHub workflow
Thoughts or suggestions?
The text was updated successfully, but these errors were encountered: