-
Notifications
You must be signed in to change notification settings - Fork 16k
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 for the spatial geometry datatype for databases #6957
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
sorry what exactly does this do?
regardless, we almost certainly do not want to add as a required dependency
@aksharahegde Hi , could you, please, resolve the merging issues and address the last comments (if needed)? After that, ping me and I push this PR for the review. Thanks! |
Sure @leo-gan. I will check for adding optional dependency and update the PR. Thanks |
Closing because the PR wouldn't line up with the current directory structure of the library (would need to be in /libs/langchain/langchain instead of /langchain). Feel free to reopen against the current head if it's still relevant! |
how can i help here? |
Description
Support for the spatial geometry datatype for databases while creating embedding in SQLDatabaseChain.
Issue: #5799
Dependencies
Added GeoAlchemy2 library
@dev2049