-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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: add schema-linking awel example #1081
Conversation
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.
r+
Thanks for your PR at first, but i noticed that you created a new Schemalinking will be a very critical capability in Don’t merge for now, please. @Aries-ckt |
Thanks for your suggestion, I will revise it again @csunny |
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.
r+
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.
r+
Description
feat: add schemalinking awel example:nl -> schema -> sql -> chart
How Has This Been Tested?
pre-requirements:
1. install openai python sdk
pip install -e ."[openai]"
2. set openai key and base
export OPENAI_API_KEY={your_openai_key} export OPENAI_API_BASE={your_openai_base}
or
import os os.environ["OPENAI_API_KEY"] = {your_openai_key} os.environ["OPENAI_API_BASE"] = {your_openai_base}
3. run
python examples/awel/simple_nl_schema_sql_chart_example.py
Examples:
Snapshots:
Checklist: