Skip to content
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

Add E2B AI Analyst #768

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

jamesmurdza
Copy link

What does this PR do?

This PR adds an AI data analyst that uses Llama APIs or Ollama to analyze data and generate charts. It uses Llama to generate the Python code and E2B to run the actual code.

A live demo is available here: https://ai-analyst.e2b.dev/

Example-ezgif com-video-to-gif-converter

The app is a self contained NextJS project that can be easily self-hosted on Vercel and integrated with API providers and Ollama via the .env file.

We have also signed the CLA.

@HamidShojanazeri
Copy link
Contributor

Thanks @jamesmurdza for the PR! its a cool recipe, if there is no hard dependency, can we please have short readme + the online version link instead of the submodule.

@jamesmurdza
Copy link
Author

Great idea, done. Let me know if I should make the README shorter or change anything.

@tizkovatereza
Copy link

Hello @HamidShojanazeri , is there anything else to update on our (E2B) end for this PR to be merged? Thanks for update :)
T.

@HamidShojanazeri
Copy link
Contributor

Thanks @tizkovatereza LGTM, can you pls fix the spellcheck errors, by adding those words to here, https://github.com/meta-llama/llama-recipes/blob/main/.github/scripts/spellcheck_conf/wordlist.txt.

@mishushakov
Copy link

file: recipes/3p_integrations/e2b-ai-analyst/README.md
words:
TogetherAI
Vercel's
echarts

@jamesmurdza
Copy link
Author

Done.

@jamesmurdza
Copy link
Author

Hey the CI seems to be failing on a Markdown link that we didn't add. Is there anything we should fix?

@tizkovatereza
Copy link

Hey @HamidShojanazeri could you please check the Mardown link as James says? We would love to see this PR merged and make an announcement. :)

Thank you very much!
T

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.

5 participants