-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Feat/mymagicai #10750
Feat/mymagicai #10750
Conversation
fix: remove python example scripts from mymagic directory
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
llama-index-integrations/llms/llama-index-llms-mymagic/llama_index/llms/mymagic/base.py
Show resolved
Hide resolved
llama-index-integrations/llms/llama-index-llms-mymagic/llama_index/llms/mymagic/base.py
Show resolved
Hide resolved
docs/examples/llm/mymagic.ipynb
Outdated
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.
The notebook has a few error outputs
You can install rhe package with pip install -e <path to root dir of mymgaic>
and then ensure your notebook venv us using the same env where you installed it.
If you can't get that to work, let's remove the notebook outputs altogether
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.
Ok, I will try to make it work with venv and if it fails, I will remove outputs
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.
Actually, since the users need to use their personal access token to run the notebook, there is no point to show any output. Let's opt in for the second option, i.e. removing outputs from the notebook.
I'm not able to contribute to this PR, but are you able to install pants and run (We need to update the CLI package init tool to insert these 😅) |
Sure I will install and run pants |
Description
MyMagic AI provides batch inference at scale. This PR integrates (wraps) MyMagic AI's API into existing LlamaIndex integrations.
Type of Change
How Has This Been Tested?
Suggested Checklist:
make format; make lint
to appease the lint gods