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

feat: added new 'hint' wrappers that inject hints into the pre-prefix #707

Merged
merged 3 commits into from
Dec 25, 2023
Merged

feat: added new 'hint' wrappers that inject hints into the pre-prefix #707

merged 3 commits into from
Dec 25, 2023

Conversation

cpacker
Copy link
Collaborator

@cpacker cpacker commented Dec 25, 2023 •
edited
Loading

Please describe the purpose of this pull request.

  • added new 'hint' wrappers that inject hints into the pre-prefix
    • chatml-hints
    • chatml-noforce-hints
  • goal is to improve proactivity memory edits / reduce hallucination of functions / improve recall of core memory

How to test

  • new wrappers tested on hosted endpoint
  • regression test old wrappers for runtime errors

Have you tested this PR?

  • regression (airoboros, chatml, chatml-noforce)
  • new wrappers (chatml-hints, chatml-noforce-hints)

@cpacker cpacker merged commit 40c62f9 into main Dec 25, 2023
5 checks passed
@cpacker cpacker deleted the new-hint-wrappers branch December 25, 2023 19:31
sarahwooders pushed a commit that referenced this pull request Dec 26, 2023
@cpacker @sarahwooders
feat: added new 'hint' wrappers that inject hints into the pre-prefix (…
7512432 
…#707)

* added new 'hint' wrappers that inject hints into the pre-prefix

* modified basic search functions with extra input sanitization

* updated first message prefix
norton120 pushed a commit to norton120/MemGPT that referenced this pull request Feb 15, 2024
@cpacker @norton120
feat: added new 'hint' wrappers that inject hints into the pre-prefix (…
9cf7e21 
…letta-ai#707)

* added new 'hint' wrappers that inject hints into the pre-prefix

* modified basic search functions with extra input sanitization

* updated first message prefix
mattzh72 pushed a commit that referenced this pull request Oct 9, 2024
@cpacker
feat: added new 'hint' wrappers that inject hints into the pre-prefix (…
399fa39 
…#707)

* added new 'hint' wrappers that inject hints into the pre-prefix

* modified basic search functions with extra input sanitization

* updated first message prefix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cpacker