-
Notifications
You must be signed in to change notification settings - Fork 221
Cannot seem to run mistral as it doesn't have a version #245
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
Comments
@fraserburch What version of the Replicate library are you using? The type definitions for these functions have supported model identifiers with and without versions since #169. replicate-javascript/index.d.ts Lines 129 to 149 in e42aa29
|
Maybe I'll manually make the change in the Thanks for the code snippet. Will give this a go! |
Sounds good! Let me know how that works for you.
I'm curious to dig into this a bit more. Replicate has only a single optional dependency, and that's a recent addition. https://github.com/replicate/replicate-javascript/blob/main/package.json#L36-L48 Can you say more about how you've tried to upgrade and what errors you found? Like, what happens if you run |
Hey Matt, We managed to update replicate module and resolve the conflicts. It wasn't replicate causing the conflicts, it was another module using replicate that was giving us grief! I'll close this ticket, but just one unrelated question. If I'm interacting with the rest api, is there anyway around the call to create prediction and then separate call to get the results? would be nice to just make one call and get back the result. Thanks! |
@fraserburch we don't have a "synchronous" api at the moment, but we are working on it. |
Seems format in docs is:
"stability-ai/sdxl:39ed52f2a78e934b3ba6e2a89f5b1c712de7dfea535525255b1aa35c5565e08b"
What about for mistral: This isn't working for me
The text was updated successfully, but these errors were encountered: