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.

Sign up for GitHub

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

Jump to bottom

Neo4j enhanced schema #20983

Merged
merged 10 commits into from Apr 29, 2024
Merged

Neo4j enhanced schema #20983

merged 10 commits into from Apr 29, 2024

Conversation

tomasonjo
Copy link
Contributor

@tomasonjo tomasonjo commented Apr 28, 2024 •
edited

Scan the database for example values and provide them to an LLM for better inference of Text2cypher

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Apr 28, 2024
@vercel Vercel
Copy link

vercel bot commented Apr 28, 2024 •
edited

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 28, 2024 9:47am

@dosubot dosubot bot added 🔌: neo4j Primarily related to Neo4j integrations 🤖:improvement Medium size change to existing code to handle new use-cases labels Apr 28, 2024
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Apr 28, 2024
@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Apr 29, 2024
@baskaryan baskaryan merged commit 67428c4 into langchain-ai:master Apr 29, 2024
62 checks passed
jexp
jexp approved these changes Apr 29, 2024
View reviewed changes
Copy link
Contributor

@jexp jexp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jexp jexp jexp approved these changes

@baskaryan baskaryan baskaryan approved these changes

Assignees
No one assigned
Labels
🤖:improvement Medium size change to existing code to handle new use-cases lgtm PR looks good. Use to confirm that a PR is ready for merging. 🔌: neo4j Primarily related to Neo4j integrations size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@tomasonjo @jexp @baskaryan