-
Notifications
You must be signed in to change notification settings - Fork 47
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
* Support more dataframes #23
Comments
It'd be good to just be able to do something like:
|
Sure. We will update it asap. |
@abhijithneilabraham keep up the great work, looking forward to contributing |
Thanks! @rhamnett The repo is updated with the changes you suggested, kindly go through the readme and examples and report any bugs. |
Hello @abhijithneilabraham! That's great! I gave it a quick test on a sample dataset i created from my own columns and the basic functionality seems to work! Some support for date types etc - datetime64 etc would be really good and then getting the model to understand what today/yesterday/this week/last week means from the current date. agent.query_db("how many website visits were there in the last week?") This kind of thing 👍 |
Makes sense. In fact, I was already building something like that, but the issue is to map the natural language to datetime without approaching it in a rule based way. Let me think about this and will update asap. |
Yes, that's whats going to make it interesting :) |
Hi, |
Haven't implemeted multi-table support yet. We need to join tables in that case, it seems. Will plan that for future. Thanks 😊 . |
thanks for the quick reply. Then df should be |
That's correct. Download a table as a dataframe and pass it to this. You could also contribute a feature if you can enable something like agent=Agent(db_url, table_name). Refer |
Sure, will check that. And send you a pr if get anything solid. |
Thanks! |
This has been done too! We have a |
Hi Abhijit |
Hi @dharmesh2002 , Can you be more specific where you have tested a scenario with SQL joins? |
Hi @abhijithneilabraham , i haven't tested sql queries, what i need is, to connect sql and than to join with tables. So that's why i was checking with you whether that is feasible? |
@abhijithneilabraham - is it possible to add new functions like group by or having clause? |
Yes. Can you add support for them?
…On Thu, Apr 21, 2022, 7:48 AM dharmesh2002 ***@***.***> wrote:
@abhijithneilabraham <https://github.com/abhijithneilabraham> - is it
possible to add new functions like group by or having clause?
—
Reply to this email directly, view it on GitHub
<#23 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIOHO46G666EDB72L7LWKMDVGC3FTANCNFSM4RZTADAQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@abhijithneilabraham sorry I didn’t understood, can u pls help me where to add group by function, I tried adding but it didn’t work |
@dharmesh2002 Can you make a pull request with your code and tag it here? |
|
@abhijithneilabraham please if you can have a look on pool request we shared above |
Hi, I reviewed the code. Have you tried testing it on an actual database
and see if the results suit your needs in a natural language query?
…On Fri, Apr 22, 2022, 9:27 PM dharmesh2002 ***@***.***> wrote:
@abhijithneilabraham <https://github.com/abhijithneilabraham> please if
you can have a look on pool request we shared above
—
Reply to this email directly, view it on GitHub
<#23 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIOHO44IC3HVSCJZKISVCWTVGLD6XANCNFSM4RZTADAQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
-Currently the feature supports csv files only. However, integrating more dataframes is easy. Go through the get_dataframe() method in data_utils.py and include support to detect the incoming file and parse the dataframe from it.
The text was updated successfully, but these errors were encountered: