Improper duplicate name validation in filter_images() about pix-plot HOT 5 CLOSED

Please review PR #271 and let me know if you need additional changes.

(P.S. Thank you all for taking the time to create and improve this project!)

from pix-plot.

Yes this has been a challenge in the past.

When we copy the filenames to the output directory, do we use just the filename or the full path?

If the latter, we should also use the full path when deduping just as you suggest. If we copy just the basename, we'll need to use the full path when copying the images to the output (like replacing / with - or similar).

from pix-plot.

When we copy the filenames to the output directory, do we use just the filename or the full path?

You use just the filename when you copy the files. The filename are used in other areas such as the imagelist (and other places).

If the latter, we should also use the full path when deduping just as you suggest...

I am not suggesting we use the full path. Although the ideal solution might be to modify the code to allow the use of files with the same name, that is a more involved process.

For now I am only suggesting we modify the validation to catch duplicate basenames with different paths.

from pix-plot.

Ah roger that--makes sense. If you'd want to send up a diff with the code you posted when you created this issue, we can get it merged!

(P.S. Thank you for your help with the project--the team has scattered and resources are limited right now, but it's great to improve the software when there are easy wins to be had!)

from pix-plot.

Issues solved in PR #271

from pix-plot.