Skip to content

feat: added Bedrock and Mistral to exec info #680

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

Merged

Conversation

LorenzoPaleari
Copy link
Contributor

More modular LLM callback created.
Added possibility to update and add additional llm model costs to a simple dictionary.

OpenAI and Bedrock handled with proprietary methods whenever possible.
Every other model will got rough a custom general callback manager.

!!! LICENSE !!!

It is possible that you need to update your LICENCE since I used and modified some code from langchain. I'll leave it up to you, but something as follows should be fine:

YOUR LICENSE

ScrapeGraphAI uses code from the Langchain
frameworks. You find their original licenses below.

THEIR LICENSE
https://github.com/langchain-ai/langchain/blob/master/LICENSE

Copy link
Collaborator

@VinciGit00 VinciGit00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you. I will also change the license

@VinciGit00 VinciGit00 merged commit 95a5ee2 into ScrapeGraphAI:pre/beta Sep 18, 2024
Copy link

🎉 This PR is included in version 1.20.0-beta.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 1.21.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants