theodo-group
DeepSeek Integration
It seems really interesting, thank you! Would you also like to add some notes to the README and to the feature matrix?
Okay, I have added usage instructions and feature description.
Hey @lyhsblog . Thanks for the PR! Quick question: did you try to extend the OpenAI class like we do with Mistral? Is there a limitation doing so?
I'm asking because it would make the maintenance way easier.
I only implemented the interface and did not extend the OpenAI class. Do I need to do the same as Mistral? I don't even know what Mistral is.
On Tue, Jun 24, 2025, 17:28 Thoonsen Maxime @.***> wrote:
MaximeThoonsen left a comment (LLPhant/LLPhant#348) https://github.com/LLPhant/LLPhant/pull/348#issuecomment-2999538110
Hey @lyhsblog https://github.com/lyhsblog . Thanks for the PR! Quick question: did you try to extend the OpenAI class like we do with Mistral? Is there a limitation doing so?
I'm asking because it would make the maintenance way easier.
— Reply to this email directly, view it on GitHub https://github.com/LLPhant/LLPhant/pull/348#issuecomment-2999538110, or unsubscribe https://github.com/notifications/unsubscribe-auth/AICLMNWKYHCXJCNP7Q3IT3D3FEK3ZAVCNFSM6AAAAAB7C33GJGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDSOJZGUZTQMJRGA . You are receiving this because you were mentioned.Message ID: @.***>
Hey @lyhsblog , what do you think of a solution like this one? https://github.com/LLPhant/LLPhant/pull/359
