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

Improvement/prompt and memory #4053

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

Conversation

k-boikov
Copy link
Contributor

@k-boikov k-boikov commented May 9, 2023

Background

Improve Prompt and Summary Memory. Try to beat challenges.
Currently managed to beat B on lvl 5
image

Changes

Probably need to split each in different PRs:

  • Switch from "system" role to "user" for all command results for more weight
  • Switch from "system" role to "user" for Summary Memory
  • Removed all "user" trigger prompts in the final message history list except for the final one in order to save tokens
  • Remove Summary Memory when Nothing New has Happened
  • Reword Summary Memory prompt, split it to 4 messages
  • Increased memory challenge time and max level
  • Mock input for memory challenge so you don't wait for the timeout

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

@vercel
Copy link

vercel bot commented May 9, 2023

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
docs ⬜️ Ignored (Inspect) Visit Preview May 10, 2023 7:21pm

@vercel vercel bot temporarily deployed to Preview May 9, 2023 13:02 Inactive
@github-actions github-actions bot added the size/l label May 9, 2023
@github-actions github-actions bot added size/xl and removed size/l labels May 10, 2023
@github-actions
Copy link
Contributor

This PR exceeds the recommended size of 200 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label May 13, 2023
@github-actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

@Pwuts
Copy link
Member

Pwuts commented Sep 8, 2023

Is this worth fixing up? Having better summarization is still relevant. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflicts Automatically applied to PRs with merge conflicts size/xl
Projects
Status: 📋 Backlog
Development

Successfully merging this pull request may close these issues.

None yet

2 participants