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

Commands Being Sent To Prompt Include Full List Not Only Enabled #206

Closed
8 of 17 tasks
Electrofried opened this issue May 3, 2023 · 2 comments
Closed
8 of 17 tasks
Labels
needs triage Needs labels assigned. type | report | bug Confirmed bug in source code.

Comments

@Electrofried
Copy link
Contributor

Description

When sending prompts through to the provider (tested with Ooga, none others) the full list of commands is being sent through, not just those that are enabled.

Steps to Reproduce the Bug

Start task with a subset of commands enabled.
Check prompt being sent to API.
See all commands on the list, not just the enabled ones.

Expected Behavior

Only Enabled commands should be added to the prompt.

Actual Behavior

All commands are added even disabled ones.

Additional Context / Screenshots

No response

Operating System

  • Microsoft Windows
  • Apple MacOS
  • Linux
  • Android
  • iOS
  • Other

Python Version

  • Python <= 3.9
  • Python 3.10
  • Python 3.11

Environment Type - Connection

  • Local
  • Remote

Environment Type - Container

  • Using Docker
  • Not Using Docker

Acknowledgements

  • My issue title is concise, descriptive, and in title casing.
  • I have searched the existing issues to make sure this bug has not been reported yet.
  • I am using the latest version of Agent-LLM.
  • I have provided enough information for the maintainers to reproduce and diagnose the issue.
@Electrofried Electrofried added needs triage Needs labels assigned. type | report | bug Confirmed bug in source code. labels May 3, 2023
@Josh-XT
Copy link
Owner

Josh-XT commented May 3, 2023

Fixed in #207 for instruct at least, still an issue?

@Josh-XT Josh-XT closed this as completed May 3, 2023
@Josh-XT Josh-XT reopened this May 3, 2023
@Josh-XT
Copy link
Owner

Josh-XT commented May 3, 2023

Found was still broken in execution agent and fixed there too in #220

@Josh-XT Josh-XT closed this as completed May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage Needs labels assigned. type | report | bug Confirmed bug in source code.
Projects
None yet
Development

No branches or pull requests

2 participants