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

[Prompt Fix] Make prompt less confused with transaction naming #31

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

ryanleung
Copy link
Contributor

@ryanleung ryanleung commented Feb 26, 2024

Context:

  • Currently the agent can be confused with the prompt because we have defined a "transaction" type within our code. This coincides with a Metamask transaction. This is causing confusion for queries like "What is my balance?", where the prompt does not think we should do anything because it is not a Metamask transaction (i.e. it doesn't transfer payment), but it is a "transaction" within our code (current transactions we define are "Transfer" and "Balance"), so it returns a balance of 0.

For this PR, I've

  • Reformatted the prompt so it's cleaner to add new information
  • Renamed the "transaction" keyword to "action", because Metamask transactions are different from how we referred to transactions in the code. This was causing some confusion for the agent (e.g. asking for a balance isn't a transaction in Metamask, but it was defined as a transaction in code).

Here are screenshots:

Without this change (see how there is no transaction field filled out. The transaction field should say "balance"):

Screenshot 2024-02-26 at 2 30 33 PM

With this change (we successfully report the balance that I have, 0.01 Eth) :

Screenshot 2024-02-26 at 2 32 19 PM

I've confirmed other behavior works the same.

@betterbrand betterbrand merged commit ec19391 into MorpheusAIs:development Feb 27, 2024
3 checks passed
@ryanleung ryanleung deleted the ryanleung-action branch February 27, 2024 03:17
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