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

feat(agent, forge): Markdown-formatted history -> message history #7228

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

Conversation

Pwuts
Copy link
Member

@Pwuts Pwuts commented Jun 18, 2024

Atomic, master-based version of #7223

Changes

  • Implement message based history in ActionHistoryComponent
  • Run ActionHistoryComponent after SystemComponent so that history messages are last in the prompt
  • Omit final instruction message if prompt already contains assistant messages

Also:
- Run `ActionHistoryComponent` after `SystemComponent` so that history messages are last in the prompt
- Omit final instruction message if prompt already contains assistant messages
@Pwuts Pwuts requested a review from a team as a code owner June 18, 2024 21:45
@Pwuts Pwuts requested review from Torantulino and majdyz and removed request for a team June 18, 2024 21:45
Copy link

netlify bot commented Jun 18, 2024

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 2797b7c
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/667a432635221c0008a99bac

@Pwuts Pwuts mentioned this pull request Jun 18, 2024
Copy link

codecov bot commented Jun 18, 2024

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 25.50%. Comparing base (c19ab2b) to head (2797b7c).
Report is 4 commits behind head on master.

Files Patch % Lines
autogpt/autogpt/agents/agent.py 0.00% 0 Missing and 1 partial ⚠️
...togpt/autogpt/agents/prompt_strategies/one_shot.py 0.00% 1 Missing ⚠️
forge/forge/agent/forge_agent.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7228      +/-   ##
==========================================
- Coverage   25.50%   25.50%   -0.01%     
==========================================
  Files          80       80              
  Lines        4661     4662       +1     
  Branches      631      631              
==========================================
  Hits         1189     1189              
- Misses       3402     3403       +1     
  Partials       70       70              
Flag Coverage Δ
Linux 25.50% <0.00%> (-0.01%) ⬇️
Windows 25.49% <0.00%> (-0.01%) ⬇️
autogpt-agent 35.91% <0.00%> (-0.03%) ⬇️
forge 21.63% <0.00%> (ø)
macOS 25.50% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Jun 19, 2024
Copy link
Contributor

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

@github-actions github-actions bot added AutoGPT Agent Forge and removed conflicts Automatically applied to PRs with merge conflicts labels Jun 19, 2024
Copy link
Contributor

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

@Pwuts Pwuts force-pushed the reinier/open-1301-change-from-markdown-formatted-history-to-message-history branch 2 times, most recently from be9daed to 2797b7c Compare June 25, 2024 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🆕 Needs initial review
Development

Successfully merging this pull request may close these issues.

None yet

2 participants