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

Wip/fengsh/merge upstream 0821 #36

Merged
merged 3 commits into from
Aug 21, 2024
Merged

Conversation

fengsh27
Copy link
Owner

No description provided.

fengsh27 and others added 3 commits August 21, 2024 00:47
* fix some bugs in reflexion agent

* fix an vectorstore issue

* introduce use_reflexion to kg RagAgent

* prompt LLM not to generate cypher marker

* fix test error

* change vectorstore collection names to apply previous fix of collection field max length issue

* phrasing, formatting, parameter explanations

* fix type hinting

* prompt LLM even if kg query result is empty

---------

Co-authored-by: fengsh <[email protected]>
Co-authored-by: slobentanzer <[email protected]>
@fengsh27 fengsh27 merged commit 715d09e into main Aug 21, 2024
1 check passed
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.

2 participants