Skip to content

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

Closed
fraserburch opened this issue Apr 23, 2024 · 5 comments
Closed

Cannot seem to run mistral as it doesn't have a version #245

fraserburch opened this issue Apr 23, 2024 · 5 comments

Comments

@fraserburch
Copy link

Screenshot 2024-04-22 at 9 22 18 PM

Seems format in docs is: "stability-ai/sdxl:39ed52f2a78e934b3ba6e2a89f5b1c712de7dfea535525255b1aa35c5565e08b"
What about for mistral: This isn't working for me

@mattt
Copy link
Contributor

mattt commented Apr 23, 2024

@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.

run(
identifier: `${string}/${string}` | `${string}/${string}:${string}`,
options: {
input: object;
wait?: { interval?: number };
webhook?: string;
webhook_events_filter?: WebhookEventType[];
signal?: AbortSignal;
},
progress?: (prediction: Prediction) => void
): Promise<object>;
stream(
identifier: `${string}/${string}` | `${string}/${string}:${string}`,
options: {
input: object;
webhook?: string;
webhook_events_filter?: WebhookEventType[];
signal?: AbortSignal;
}
): AsyncGenerator<ServerSentEvent>;

@fraserburch
Copy link
Author

Maybe I'll manually make the change in the index.d.ts file!
My version is definitely outdated, and I haven't had any luck updating it due to dependency conflicts!

Thanks for the code snippet. Will give this a go!

@mattt
Copy link
Contributor

mattt commented Apr 23, 2024

Maybe I'll manually make the change in the index.d.ts file!

Sounds good! Let me know how that works for you.

My version is definitely outdated, and I haven't had any luck updating it due to dependency conflicts!

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 npm i replicate@latest?

@fraserburch
Copy link
Author

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!

@aron
Copy link
Contributor

aron commented May 2, 2024

@fraserburch we don't have a "synchronous" api at the moment, but we are working on it.

@aron aron closed this as completed May 2, 2024
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

No branches or pull requests

3 participants