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: Image descriptions into interaction.ts #1775

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

denizekiz
Copy link
Contributor

@denizekiz denizekiz commented Jan 3, 2025

Relates to:

Risks

low

Background

Image descriptions was missing, added image descriptions and injected into prompts through state and context.

What does this PR do?

Add ability to agent to see images that they are tagged.

What kind of change is this?

Features (non-breaking change which adds functionality)

Documentation changes needed?

My changes do not require a change to the project documentation.

Testing

tag your twitter account with an image
pnpm build
pnpm start

Detailed testing steps

Screenshots

ss

@wtfsayo wtfsayo requested review from wtfsayo and shakkernerd January 8, 2025 11:05
Copy link
Collaborator

@monilpat monilpat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM outside of conflicts

@@ -342,11 +348,36 @@ export class TwitterInteractionClient {

elizaLogger.debug("formattedConversation: ", formattedConversation);

//Get image description for the provided iamges.
//console.log(tweet);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can remove this

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.

4 participants