-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
@@ -1,6 +1,6 @@ | |||
from __future__ import print_function | |||
from csv import reader, DictReader | |||
import exifread as er | |||
# import exifread as er |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just delete the entire line
@kdmurray91 Should be good to go! |
@@ -236,24 +249,109 @@ def __call__(self, x): | |||
return None | |||
|
|||
|
|||
def resize_img(filename, to_width): | |||
# Open the Image and get its width | |||
if(SKIMAGE): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd do
if not SKIMAGE:
warnings.warn('blah')
return
rest
of
the
code
@kdmurray91 Made all changes, ready for pull |
try: | ||
from skimage.transform import rescale | ||
from skimage import novice | ||
import skimage.io as io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd be keener to do import skimage
and use skimage.io.whatever
, as there's a stdlib io module.
likewise with novice
and rescale
above
@jbraiuka looks great, few more minor comments. Mostly pedantry. Ping me once those are done and I'll merge. |
@kdmurray91 Done. |
Perfect. One last change, but I'll fix when I merge on the command line |
@jbraiuka all done, nice work man. I'll make a new release and upload to PyPI now. |
Able to scrape the date from the filename and write this to the Exif Data of the Image.
Also changed from using ExifRead to pexif (Which has read and write capabilities)