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

community: update Memgraph integration #27017

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

katarinasupe
Copy link
Contributor

@katarinasupe katarinasupe commented Oct 1, 2024

Description:

  • Memgraph no longer relies on Neo4jGraphStore but implements GraphStore, just like other graph databases.
  • Memgraph no longer relies on GraphQAChain, but implements MemgraphQAChain, just like other graph databases.
  • The refresh schema procedure has been updated to try using SHOW SCHEMA INFO. The fallback uses Cypher queries (a combination of schema and Cypher) → LangChain integration no longer relies on MAGE library.
  • The schema structure has been reformatted. Regardless of the procedures used to get schema, schema structure is the same.
  • The add_graph_documents() method has been implemented. It transforms GraphDocument into Cypher queries and creates a graph in Memgraph. It implements the ability to use baseEntityLabel to improve speed (baseEntityLabel has an index on the id property). It also implements the ability to include sources by creating a MENTIONS relationship to the source document.
  • Jupyter Notebook for Memgraph has been updated.
  • Issue: /
  • Dependencies: /
  • Twitter handle: supe_katarina

I initially had all imports in langchain package as well, since I followed the implementation of others, but now I removed it based on the PR template.

I also ran make format and make lint, but based on the PR template, I didn't want to commit changes on other integrations. Let me know if that's okay.

Copy link

vercel bot commented Oct 1, 2024

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 Oct 1, 2024 0:18am

@katarinasupe katarinasupe changed the title Memgraph integration community: update Memgraph integration Oct 1, 2024
@katarinasupe katarinasupe marked this pull request as ready for review October 1, 2024 11:52
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. community Related to langchain-community 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder labels Oct 1, 2024
@katarinasupe
Copy link
Contributor Author

Hi @baskaryan, tagging as noted in PR template. Please let me know what I need to do regarding the test and what's needed to do in order to get this merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Related to langchain-community 🤖:docs Changes to documentation and examples, like .md, .rst, .ipynb files. Changes to the docs/ folder size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
Status: Triage
Development

Successfully merging this pull request may close these issues.

1 participant