-
Notifications
You must be signed in to change notification settings - Fork 6
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
Check git remote exists #126
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @dberenbaum !
We should probably still review the error message on the Studio side though ... it's not clear why it was mentioning gitlab self-hosted ... it's a low priority clearly |
AFAIR Studio tries to validate the URL format, but there wasn't always an easy way to identify all self-hosted gitlab URL formats, so it is a catchall that assumes any URL it can't validate is self-hosted gitlab. |
Fixes iterative/dvclive#753.
Before this PR:
After this PR:
$ python train.py WARNING:dvc_studio_client:No Git remote. Can't infer Studio repo URL.