Allow images with upper case file extensions. e.g. foo.JPG

[jim-meyer]

[shensheng272]

Thanks for pr. But currently IMG_FORMAT is used with .lower() . So images with upper case file extensions will cause no trouble.

[jim-meyer]

@shensheng272 There is a bug here where the code is not using .lower(). So either that line needs to be changed to call .lower(), or IMG_FORMAT needs to include both upper and lower case versions of the supported extensions. I though changing IMG_FORMAT was a more robust solution to avoid future problems like this. Would you rather I submit a PR for fixing that line of code instead?

There is a similar problem here too hence my suggested change to that file.

[shensheng272]

@shensheng272 There is a bug here where the code is not using .lower(). So either that line needs to be changed to call .lower(), or IMG_FORMAT needs to include both upper and lower case versions of the supported extensions. I though changing IMG_FORMAT was a more robust solution to avoid future problems like this. Would you rather I submit a PR for fixing that line of code instead?

There is a similar problem here too hence my suggested change to that file.

Thanks for reply. Your example of vis_dataset is right. You might submit a PR for fixing that line of code of vis_dataset.py instead. Sorry for my careless check.