restore "CZICmd" demo application

[ptahmose]

Description

This is adding the sample-application "CZICmd". It is functionally identical with the
version from here, with the following change: the command-line argument parsing has been refactored and is now done with the help of CLI11.

Note: this PR is just adding the code, but CZICmd is not built by default. It needs to be enabled by setting the option "LIBCZI_BUILD_CZICMD" to "ON".

Fix for #18 is included.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

• build runs locally (Linux and Windows)
• CZICmd tested manually

Checklist:

• I followed the Contributing Guidelines.
• I did a self-review.
• I commented my code, particularly in hard-to-understand areas.
• I updated the documentation.
• I have added tests that prove my fix is effective or that my feature works.
• New and existing unit tests pass locally with my changes.

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (main@7b425bd). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #20   +/-   ##
=======================================
  Coverage        ?   46.20%           
=======================================
  Files           ?       77           
  Lines           ?    15659           
  Branches        ?        0           
=======================================
  Hits            ?     7235           
  Misses          ?     8424           
  Partials        ?        0           

Flag	Coverage Δ
windows-latest	46.20% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[FelixS90]

Reported code coverage is OK -> 46.2% is the same as currently on main. No comparison to base commit since branch seems to have been based on commit prior to introducing codecov.

[FelixS90]

Note: Not further reviewing the logic of Src/CZICmd based on PR description stating that this is basically "copied" over from the archived repo and functionally equivalent.

[FelixS90]

Seems that Src/libCZI/Doxyfile was never used?

I'd think so, only ..Src/Doxyfile is used.

[FelixS90]

Please have a look at the open conversations/threads. Looks good to me.

[FelixS90]

LGTM. Just a few open comments.

[FelixS90]

One thing I just noticed when running doxygen locally (version 1.9.5):

Maybe we want to remove the unused flags. I leave this up to you.

Info: The last successful run on main was still using 1.8. Most likely, it will then also use 1.9 next time (i.e. upon completion of this PR), but as I said, we can also tackle this later on when we get the warning on the CD build.

[FelixS90]

What are your plans on adding tests for CZICmd?

Should we still already include it in https://github.com/ZEISS/libczi/blob/main/opencppcoverage.txt (to have visibility through codecov)?

Or should we add an issue (maybe tagged "good first issue") to keep track of it?

What do you think?

[ptahmose]

One thing I just noticed when running doxygen locally (version 1.9.5):

Maybe we want to remove the unused flags. I leave this up to you.

Info: The last successful run on main was still using 1.8. Most likely, it will then also use 1.9 next time (i.e. upon completion of this PR), but as I said, we can also tackle this later on when we get the warning on the CD build.

One thing I just noticed when running doxygen locally (version 1.9.5):

Maybe we want to remove the unused flags. I leave this up to you.

Info: The last successful run on main was still using 1.8. Most likely, it will then also use 1.9 next time (i.e. upon completion of this PR), but as I said, we can also tackle this later on when we get the warning on the CD build.

added an issue -> #21

[ptahmose]

What are your plans on adding tests for CZICmd?

Should we still already include it in https://github.com/ZEISS/libczi/blob/main/opencppcoverage.txt (to have visibility through codecov)?

Or should we add an issue (maybe tagged "good first issue") to keep track of it?

What do you think?

Added #24 and discussed ideas/options.
For the time being, I'd be fine with "codecov does not include CZIcmd".