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

Re-Implemented Load Endpoint Plus Other Adjustments #30

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zach-blumenfeld
Copy link

Having a way to load example datasets is super helpful for people exploring, learning, and demoing. As such I re-implemented the kg_chat load endpoint that was taken out in #29 and updated neo4j connection setup as a fix to accommodate writes for the endpoint. Mixed in a couple of other small changes. Detailed changes below:

  • Re-implemented init and load GET endpoints for kg_chat backend. Providing starter datasets is super helpful for others demoing.
    • Separated a read and write neo4j connection to allow loading while keeping other endpoint read-only.
  • Small updates to kg_chat Readme for clearer demo directions
  • Updated neo4j version 5.7 -> 5.9 in docker-compose
  • A few minor code formatting fixes and .gitignore addition

- Small updates to kg_cha` Readme for clearer demo directions
- Re-implemented `init` and `load` GET endpoints for kg_chat backend. Providing starter datasets is super helpful for others demoing.
- Separated a read and write neo4j connection to allow loading while keeping other endpoint read-only.
- Updated neo4j version 5.7 -> 5.9 in docker-compose
- A few minor code formatting fixes and .gitignore addition
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 this pull request may close these issues.

1 participant