openagi icon indicating copy to clipboard operation
openagi copied to clipboard
Published 9 months ago • verified publisher icon aiplanethub

Removing langchain api and replacing with original api for all llms

Open AdvH039 opened this issue 1 year ago • 6 comments

Refer #79

AdvH039 avatar Oct 26 '24 08:10 AdvH039

@tarun-aiplanet please let me know you're happy with the implementation and if I can proceed the same with other llms. Thank you.

AdvH039 avatar Oct 27 '24 10:10 AdvH039

Hi @AdvH039 Thanks for the PR. This is something similar we need. I will test it for Groq. Can you add for 2 more LLMs, so that we can test it at same time for 3 LLMs

tarun-aiplanet avatar Oct 28 '24 06:10 tarun-aiplanet

@tarun-aiplanet I added two more llms and used try-except. Please let me know if it's working. Thanks.

AdvH039 avatar Oct 28 '24 14:10 AdvH039

@tarun-aiplanet I added two more llms and used try-except. Please let me know if it's working. Thanks.

cool. this looks good. the import statement and system prompt needs to fixed.

tarun-aiplanet avatar Oct 28 '24 14:10 tarun-aiplanet

@tarun-aiplanet Done. Thanks.

AdvH039 avatar Oct 28 '24 20:10 AdvH039

@AdvH039 Congrats on being approved for the Hacktoberfest swags! Please share your details with me so we can send it over to you. You can either drop the details in our Discord channel: bit.ly/aiplanet-discord or email me at [email protected].

shivaya-aiplanet avatar Nov 20 '24 09:11 shivaya-aiplanet