Jaisen Mathai

Results 224 comments of Jaisen Mathai

@WilfriedBl there have been various unicode errors but if you share your photo I can see if it fails.

We made [this change](https://github.com/jmathai/elodie/commit/fe70227c7170e01c8377de7f9770e761eab52036) to `pyexiftool.py` which may help.

Seems like the `json.loads` line does throw the exception. Unsure if it's the same as what the commit I pointed to fixes though. ``` File "D:\PyCharmProjects\Daminion\exiftool.py", line 315, in execute_json...

@vkhobor were you able to make a code that was more generic which worked for you?

Can you update [this line of code](https://github.com/jmathai/elodie/blob/master/elodie/filesystem.py#L173)? ``` this_value = re.sub(self.whitespace_regex, '-', str(metadata[part]).strip()) ```

Great to hear :). Reopening until we can have it fixed via a pull requests - feel free to submit :)

@bdillahu understandable about tests. I'll see if I can get to tests as well. Always appreciate PRs submitted early so they can be iterated upon.

Hi @alex-phillips, we should not be changing the `ctime` as far as I know. We're only setting the `atime` and `mtime`. Here's [the source](https://github.com/jmathai/elodie/blame/master/elodie/filesystem.py#L590) with blame. We're calling [os.utime](https://docs.python.org/2/library/os.html#os.utime). That...

Thanks for this PR. I'm not sure I understand how this interacts with #310 which attempts to utilize location data to determine the proper time zone. Feel free to describe...