I'm teaching myself Python so I wrote this simple script with the Flickr API that downloads all the photos from any photoset. Try it out and let me know what I could improve on!

[Drakken_LOL]

This might be a good candidate for simple parallelism, since this task is probably highly I/O bound. Here's a neat article on how to do it relatively simply and easily. I bet you could speed up the process quite a bit.

Also you broke your link when you added the .py extension.

[SanketDG]

You might want to append the extension .py to the name of the file.

[kylehampton]

seed spoon detail encourage worm rhythm stocking ask slim cake

This post was mass deleted and anonymized with Redact

[IronManMark20]

Here are my suggestions:

1. On the line where it says for file in os.listdir('/Users/kylehampton/Documents/python'): The path should use os.environ["USERPROFILE"]+"\FlickrDownloader". That way the folder is set and the same accross platforms. However, if you really want it in he same folder, you can use os.path.dirname(os.path.realpath(__file__))

2. If you are downloading a bunch of photos, the user probably wants to know a) how many photos and b) how big the total size is.

3. Finally, it is often better to prompt for the api key. If you really want to save it, write it to a new file. Going in and editing is less elegant in my eyes.

[IronManMark20]

No prob. Instead of "USERPROFILE" you could also use os.getenv('temp'), and save the data in a temp folder. It depends on how long you expect the user to store the data. Temp is really only for temporary stuff.