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

Use flickrapi to do the oauth stuff for us #1

Merged
merged 2 commits into from Oct 8, 2013

Conversation

Projects
None yet
2 participants
@hugovk
Contributor

hugovk commented Oct 8, 2013

flickrapi from here: http://stuvel.eu/flickrapi

4up-photos.py uses a flickrapi utility to walk through search results. http://stuvel.eu/media/flickrapi-docs/documentation/#walking-through-a-search-result

4up-faves.py had a bug in the while loop (for me) because it compares an int page to str pages:
while not pages or page <= pages

Fix:
while not pages or page <= int(pages)

(The while loop was replaced in 4up-photos by the photo walk.)

@hugovk

This comment has been minimized.

Show comment
Hide comment
@hugovk

hugovk Oct 8, 2013

Contributor

Tested and it creates sane-looking csv files.

layout.pl didn't work for me. First I had to replace the font filenames for Mac:

_freetype_so = ctypes.CDLL("/usr/local/lib/libfreetype.6.dylib") _cairo_so = ctypes.CDLL("/usr/local/lib/libcairo.2.dylib")

And then I got Segmentation fault: 11 so there's probably something up with my Cairo install. But that's for another time.

Contributor

hugovk commented Oct 8, 2013

Tested and it creates sane-looking csv files.

layout.pl didn't work for me. First I had to replace the font filenames for Mac:

_freetype_so = ctypes.CDLL("/usr/local/lib/libfreetype.6.dylib") _cairo_so = ctypes.CDLL("/usr/local/lib/libcairo.2.dylib")

And then I got Segmentation fault: 11 so there's probably something up with my Cairo install. But that's for another time.

straup added a commit that referenced this pull request Oct 8, 2013

Merge pull request #1 from hugovk/master
Use flickrapi to do the oauth stuff for us

@straup straup merged commit 6d20bbd into straup:master Oct 8, 2013

@straup

This comment has been minimized.

Show comment
Hide comment
@straup

straup Oct 8, 2013

Owner

Thanks!

Owner

straup commented Oct 8, 2013

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment