Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various Changes and Improvements, Additional Flags #23

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

notquitenothing
Copy link

@notquitenothing notquitenothing commented Aug 6, 2024

Various changes and improvements. Changes should all be backwards compatible and behind flags, should be almost no change in behavior unless the new flags are used. Open to feedback, feel free to merge as is, cherry-pick, or reject. Changes include:

  • Adding an output folder parameter (-OutputFolderPath,-o) to select an output folder separate from the input file.
  • Added a flag (-OutputUnconverted, -u) which will copy image files already in the converted format, jpeg, to the output folder if it is specified.
  • Added an interactive mode flag (-InteractiveMode, -t) which will open dialog file pickers for input files and output folder if they are not specified in a parameter.
  • Added a flag to allow file conversion to continue on a failed convert (-NoFailOnConvert). Files that failed to convert are printed in a message at the end of the script.
  • File logging now includes the full output file path, since it may be different from the input file.

Added intereactive mode that when active, opens dialog boxes to select input files and output folder if they are not provided.
Added flag to also copy unconverted image files, such as those that are already in jpeg format, to output folder.
When a file fails to convert, add it to a list that is printed at the end of execution instead of exiting immediately.
Conversion output now includes full output file path since it can be selected.
@notquitenothing notquitenothing changed the title Various chan and Various Changes and Improvements, Additional Flags Aug 6, 2024
@DavidAnson
Copy link
Owner

Thank you for the submission! These look like some interesting changes and you seem to have been very thorough about the process. The challenge on my side is I haven’t used this script (or PowerShell) for a few years and confidence in my ability to review effectively is low.

My head is not in the right place to accept this PR now, but I will leave it open for consideration. For your own future reference, maintainers will generally be more receptive to multiple standalone changes vs. a single one that adds unrelated features.

Thank you again for sharing this! If you decide to fork the project out of impatience, I will understand. :)

@notquitenothing
Copy link
Author

Totally understand, feel free to leave it open, close it, or copy only what you want whenever you get around to it. My changes were to make it easier for a family member to use the script, using a shortcut to populate some of the flags, and thought someone else might get some use out of it. Thanks for your work 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants