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

Convert Chapter 2 to use DDGo to smooth learning experience? #649

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sunlambda
Copy link

Synopsis

Setting up a Microsoft Azure account for learning purposes is distracting, especially for Chapter 2. There has to be a keyless way to get some image URL's...I was just talking about this in #648 so I wanted to share my result.

Problem

I followed all the instructions (in Fastbook and elsewhere), but I was still having trouble creating the Azure cloud resource to do the Bing Image search (deployment failure, BadRequest, Internal Server Error).

Expected Behavior

Do Chapter 2 without signing up for more things, and to minimize the distractions so the attention can be paid to learning PyTorch.

Observed Behavior

I spent more time trying to figure out Azure portal, than simply adjusting the Chapter 2 to use DDGS and submitting this PR! Now I can complete Chapter 2 and continue on with the book confidently.

Proposed Solution

Use a DDGS library which requires no key, with the trade off of being limited by query volume.

Content of this PR

Attached to this PR is a tested notebook which installs the referenced DDGS library. Then, any instances of search_images_bing is replaced with ddgs.images(**query_parameters), I was specific with the query parameters (small, public domain, etc.)

TODO

The literature in the Notebook itself may need more updating, but the code should work and the notebook should run.

The actual text has not necessarily been updated, and saying that the images come from Bing isn't exactly inaccurate either.
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@sunlambda sunlambda changed the title Convert to DDGS so that I can complete the chapter! Convert Chapter 2 to use DDGo to smooth learning experience? May 16, 2024
@bd4
Copy link

bd4 commented Jul 24, 2024

Thanks for this! Unfortunately I get a RateLimitException trying to execute it. I am going to try again tomorrow with some delays between API calls to see if I can get it to work.

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.

2 participants