-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Update to extend support for MonsterAPI new LLM APIs #14373
base: main
Are you sure you want to change the base?
Update to extend support for MonsterAPI new LLM APIs #14373
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the contribution! I have mentioned a small change also test cases needs to be updated as well
````python | ||
from llama_index.llms.monsterapi import MonsterLLM | ||
|
||
llm = MonsterLLM( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small correction needed here:
base_url -> api_base
monster_api_key -> api_key
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! This is basically the only thing I noticed as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, updated! Good catch!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@logan-markewich @anoopshrma Can you guys look at the CI test failing and help me with it? we don't have monsterapi package dependency anymore. I updated the pyproject.toml accordingly not sure where the reference of monsterapi is coming in!
Description
MonsterAPI has updated its API structure to openAI API routes and synchronous APIs.
Update integration module and example colabnotebook to use with new structure.
Version Bump?
Did I bump the version in the
pyproject.toml
file of the package I am updating? (Except for thellama-index-core
package)Type of Change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Suggested Checklist:
make format; make lint
to appease the lint gods