Erick Friis

Results 66 comments of Erick Friis

Hey @gauravrele87 ! How do you hope to use this? I believe you're calling an image generation model and getting a base64-encoded image back, which doesn't fit our LLM (text-to-text)...

Hey @3coins @gauravrele87 ! Let's skip merging this into the LLM class because it doesn't make sense in the LLM abstraction. We haven't put much thought into image generation abstractions...

@ljeagle looks like the formatter/lint check failed - could you update that and I can merge it in?

Sounds good. Thanks @raoufchebri !

Closing because the PR wouldn't line up with the current directory structure of the library (would need to be in /libs/langchain/langchain instead of /langchain). Feel free to reopen against the...

Closing because the PR wouldn't line up with the current directory structure of the library (would need to be in /libs/langchain/langchain instead of /langchain). Feel free to reopen against the...

Closing because the PR wouldn't line up with the current directory structure of the library (would need to be in /libs/langchain/langchain instead of /langchain). Feel free to reopen against the...

Looks like this one needs a bit more work to land. Happy to reopen if someone wants to take it over. Otherwise I'll close for now :)

Closing because the PR wouldn't line up with the current directory structure of the library (would need to be in /libs/langchain/langchain instead of /langchain). Feel free to reopen against the...