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

SqliteConnector has no attribute host #1067

Closed
LiquidGunay opened this issue Mar 27, 2024 · 8 comments · Fixed by #1071
Closed

SqliteConnector has no attribute host #1067

LiquidGunay opened this issue Mar 27, 2024 · 8 comments · Fixed by #1071

Comments

@LiquidGunay
Copy link

System Info

OS: Ubuntu
Python: 3.10
Same error on multiple pandasai versions from 2.07 to latest

🐛 Describe the bug

When I initialise an agent with a list of SqliteConnectors (one for each table of my db), and then proceed to chat with it, it throws an attribute error: SqliteConnector has no attribute host. I suspect there is an issue in the input validation.

@gventuri
Copy link
Collaborator

@LiquidGunay it has probably be caused by the upgrade of pandas. Can you check out if the latest version works?

@LiquidGunay
Copy link
Author

No this did not fix the issue. As a workaround I modified the equals function of sql.py to not check for host and port and downgraded sqlalchemy to <2.0

@YarShev
Copy link
Contributor

YarShev commented Mar 28, 2024

@gventuri, it looks like reverting the upgrade of pandas doesn't affect this issue. Can we upgrade pandas and Modin once again?

@YarShev
Copy link
Contributor

YarShev commented Apr 1, 2024

@gventuri, just a friendly reminder on my question above. Are you okay if I submit a PR on that matter once again?

@gventuri
Copy link
Collaborator

gventuri commented Apr 2, 2024

@YarShev that was another bug (which has been fixed as part of #1071). Unfortunately we're still trying to figure out how to fixe the issue with pandas > 2 breaking the SQL connectors relying on SQLalchemy, but the upgrade to pandas 2.0 is one of the priorities.

@YarShev
Copy link
Contributor

YarShev commented Apr 2, 2024

@gventuri, ah, I see! Thanks for letting me know this. Is there an open issue for the problem you mentioned about the SQL connectors relying on SQLalchemy?

@gventuri
Copy link
Collaborator

gventuri commented Apr 2, 2024

@YarShev sure, here's the issue: #1085

@YarShev
Copy link
Contributor

YarShev commented Apr 3, 2024

@gventuri, thank you for creating an issue for the problem!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants