-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Notebook on web scraping with tool use #2192
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2192 +/- ##
=======================================
Coverage 37.94% 37.94%
=======================================
Files 77 77
Lines 7780 7780
Branches 1666 1666
=======================================
Hits 2952 2952
Misses 4579 4579
Partials 249 249
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@areibman let me know if you want to make improvement over this. |
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.
Looks good to me!
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.
Would you like to render this notebook on the website?
EDIT: It already is |
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.
In the front matter the tags use spaces instead of -
. Also scrapping
-> scraping
* Add web scrapping notebook * formatting * tags * Update * update
* Add web scrapping notebook * formatting * tags * Update * update
Why are these changes needed?
Provide an example to scrap web pages.
Related issue number
Checks