Script does not work in Python 3

[mchesterkadwell]

The following lines don't work in Python 3:
https://github.com/BritishMuseumDH/britishMuseumFacesDetection/blob/master/britishMuseumFaces.py#L259
https://github.com/BritishMuseumDH/britishMuseumFacesDetection/blob/master/britishMuseumFaces.py#L290

I think this is because the synax for using urllib changed from Python 2:
urllib.urlretrieve()
to Python 3:
urllib.request.urlretrieve()

If you have the images downloaded already you won't see this error when testing with Python 3 because it's only necessary to use urllib when fetching the images for the first time.

[portableant]

Can you try this python 3 version out? https://gist.github.com/portableant/ce270fbe9b5f1638ac01d9d2a13a6f31

[mchesterkadwell]

I have tried it out and it now works for me after I have made some changes and comments (marked with ###)
https://gist.github.com/mchesterkadwell/9c8173d11d2a821f3d4a222fd032dd5e
Three types of changes:

1. Related to Python 3 (in particular, Python 3 handles division differently than Python 2)
2. Possible bugs and some suggested solutions
3. Apparently duplicated code commented out

I am not sure if my set up has 'caused' some of these, so please cast sceptical eye.