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.

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

Better error message for non safetensor repository #15

Merged
merged 4 commits into from
May 4, 2024

Conversation

IamShubhamGupto
Copy link
Contributor

@Blaizzy
Copy link
Owner

Blaizzy commented May 4, 2024

Thanks a lot!

Let’s wait for the tests to clear.

@Blaizzy
Copy link
Owner

Blaizzy commented May 4, 2024

Could you change “local_id” to “local_dir”? And add "Then use the local_dir as the --hf-path" followed by the convert script.

After that we’ll merge it.

@Blaizzy
Copy link
Owner

Blaizzy commented May 4, 2024

@IamShubhamGupto

Great, we are almost ready!

Just missing the convert script after the last message.

@IamShubhamGupto
Copy link
Contributor Author

@IamShubhamGupto

Great, we are almost ready!

Just missing the convert script after the last message.

just added it, let me know if there are any more changes

@Blaizzy
Copy link
Owner

Blaizzy commented May 4, 2024

LGTM!

@Blaizzy Blaizzy merged commit 79c57e6 into Blaizzy:main May 4, 2024
1 check passed
@Blaizzy
Copy link
Owner

Blaizzy commented May 4, 2024

Thanks!

@Blaizzy
Copy link
Owner

Blaizzy commented May 5, 2024

@IamShubhamGupto What's your X account?

I want to tag you on the next release :)

@IamShubhamGupto
Copy link
Contributor Author

@Blaizzy haha thanks, sure my x account is @iamshubhamgupto

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FileNotFoundError: No safetensors found
2 participants