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

Resolved issue #548. made changes to agent/service and agents/views in AgentOuput and ValidationResult BaseModel. #602

Closed
wants to merge 3 commits into from

Conversation

jhachirag7
Copy link
Contributor

tool description required for bedrock, and for other models

@CLAassistant
Copy link

CLAassistant commented Feb 7, 2025

CLA assistant check
All committers have signed the CLA.

@jhachirag7
Copy link
Contributor Author

tool description required for some models added description to agent output and validation results base model

@jhachirag7 jhachirag7 changed the title Fix/tool description Resolved issue #548. made changes to agent/service and agents/views in AgentOuput and ValidationResult BaseModel. Feb 7, 2025
mbaynton pushed a commit to mbaynton/browser-use that referenced this pull request Feb 9, 2025
Full credit due to @jhachirag7 on this, I've set them as the commit
author. They also opened browser-use#602 but it looks like it has some indentation
issue (fixed here) and a few spurious commits and removed files; I only
mean to make it as easy as possible for maintainers to get this merged.

I ([email protected]) have tested that with this fix it is possible to
run against AWS Bedrock + Sonnet 3.5 v2.

Fixes browser-use#580
@MagMueller
Copy link
Collaborator

Thank you very much, merged already.

@MagMueller MagMueller closed this Feb 12, 2025
AryamanParida pushed a commit to AryamanParida/browser-use that referenced this pull request Mar 7, 2025
Full credit due to @jhachirag7 on this, I've set them as the commit
author. They also opened browser-use#602 but it looks like it has some indentation
issue (fixed here) and a few spurious commits and removed files; I only
mean to make it as easy as possible for maintainers to get this merged.

I ([email protected]) have tested that with this fix it is possible to
run against AWS Bedrock + Sonnet 3.5 v2.

Fixes browser-use#580
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.

3 participants