-
Notifications
You must be signed in to change notification settings - Fork 7k
Enable CodeAct agents with browsing, and also enable arbitrary BrowserGym action support #1807
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
Conversation
Example run log:
|
|
||
@dataclass | ||
class BrowseInteractiveAction(Action): | ||
browser_actions: str |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, I only see 'goto' browser_actions in this pr, is there any other type? Since if only one browser_action, then maybe BrowseInteractiveAction
is just the same as BrowseURLAction
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're planning on adding more actions later, but want to do it a little bit at the time as we validate that this doesn't hurt accuracy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For now, this is true, I only showed to the model in prompts about the usage of goto(). However, there are many other actions that can be supported: https://github.com/ServiceNow/BrowserGym/blob/main/core/src/browsergym/core/action/highlevel.py
Now this is to lay ground to this support in the future
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, got it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for this!
#1469 solved |
#1674