-
Notifications
You must be signed in to change notification settings - Fork 96
Browser agent notebook #250
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
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
View / edit / reply to this conversation on ReviewNB bilgeyucel commented on 2025-08-08T07:00:32Z First, we'll install all the necessary dependencies:
anakin87 commented on 2025-08-13T10:16:55Z Fixing the typo. Keeping links to docs, which are more completed and up to date (especially for MCP). |
View / edit / reply to this conversation on ReviewNB bilgeyucel commented on 2025-08-08T07:00:33Z We need to remove the TODO line here |
View / edit / reply to this conversation on ReviewNB bilgeyucel commented on 2025-08-08T07:00:33Z We add the "browser_snapshot" tool but it doesn't use it for this query, right? anakin87 commented on 2025-08-13T10:16:17Z In some runs, the Agent uses it to better understand |
View / edit / reply to this conversation on ReviewNB bilgeyucel commented on 2025-08-08T07:00:34Z Maybe link to the tutorial here |
View / edit / reply to this conversation on ReviewNB bilgeyucel commented on 2025-08-08T07:00:35Z TODO line |
View / edit / reply to this conversation on ReviewNB bilgeyucel commented on 2025-08-08T07:00:36Z Another todo reminder |
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.
I suggest:
- add videos
- Remove the text linking to demos until we have it on haystack-demos
- Merge the PR :)
In some runs, the Agent uses it to better understand View entire conversation on ReviewNB |
Fixing the typo. Keeping links to docs, which are more completed and up to date (especially for MCP). View entire conversation on ReviewNB |
This is ready for a first review.
What's missing: