Advanced search filters #1
Conversation
…l as safesearch-setting functionality.
|
Hey, I'm pretty busy this week so I probably won't have time to review/merge until the weekend. Great idea! At first glance, code looks good. Thanks for following the style :) |
talleyhoe
force-pushed
the
feature/filters
branch
from
2aaf2eb to
767c8a8
Compare
|
There's 2 big issues that have to be fixed (I'll help) 1.) The filters should all be optional 2.) Each option needs to be tested. This includes all of the choices for categorical arguments (large, medium, tall, square, etc) 3.) Plz try to avoid rows longer than 80 columns |
src/scraper.py
Outdated
| if not os.path.exists(directory): | ||
| os.makedirs(directory) | ||
|
|
||
| global search_url |
There was a problem hiding this comment.
The global should be called in setup_url so the logic is compartmentalized
src/scraper.py
Outdated
| else: | ||
| return '' | ||
|
|
||
| def process_safesearch(val: str): |
There was a problem hiding this comment.
Is the default safesearch behavior any different from "off"?
src/scraper.py
Outdated
| if (subfeatures[1] != []): | ||
| features += ["safe=" + delim2.join(subfeatures[1])] | ||
|
|
||
| print(delim1.join(features)) |
There was a problem hiding this comment.
Please wrap debugging print statements in an 'if DEBUG' and provide them useful text to identify the output
src/main.py
Outdated
| def main(): | ||
| args = get_arguments(sys.argv) | ||
| scrape_images(args.keyword[0], args.count, args.directory, args.threads, args.size, args.color, args.imgtype, args.safesearch) | ||
| scrape_images(args.keyword[0], args.count, args.directory, args.threads, args.size, args.aspectratio, args.color, args.imgtype, args.region, args.filetype, args.usage, args.safesearch) |
There was a problem hiding this comment.
Filters should be passed as a single object so we can try to avoid these long definitions that look ugly. IMO if you can't call the function on one line, it needs to be refactored into a more compact structure
|
Sorry, haven't revised/added anything, but realized that this can't be merged until it's "marked as ready". |
2 participants
Added a bunch of arguments corresponding to the various advanced image search filters and the safesearch setting.