Use Argparse in mdpow-pcw and mdpow-pow and add loggers

[VOD555]

fix #147
-replace optparse with argparse
-add loggers to print out settings

[orbeckst]

We don't have argparse in Python 2 https://docs.python.org/3/library/argparse.html so I'll leave it open until we get rid of Python 2.7

[VOD555]

@orbeckst It's in Python 2.7 https://docs.python.org/2.7/library/argparse.html

[orbeckst]

Oh, I only saw the note in the 3.x page that said “added in 3.3”. That’s good then.

…

Am 7/9/21 um 19:59 schrieb Shujie Fan ***@***.***>:  @orbeckst It's in Python 2.7 https://docs.python.org/2.7/library/argparse.html — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

[codecov]

Codecov Report

Merging #150 (1f8ee28) into develop (6a85735) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #150   +/-   ##
========================================
  Coverage    68.30%   68.30%           
========================================
  Files           10       10           
  Lines         1382     1382           
  Branches       167      167           
========================================
  Hits           944      944           
  Misses         365      365           
  Partials        73       73           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a85735...1f8ee28. Read the comment docs.

[orbeckst]

ping me when you need a review

[VOD555]

Current code cannot deal with --SI and --no-SI correctly. I'll use Argument groups for them.https://docs.python.org/3/library/argparse.html#argparse.ArgumentParser.add_mutually_exclusive_group

[VOD555]

@orbeckst This PR can be reviewed.

[orbeckst]

Minor doc corrections to do, which I'll do before merging.

Looking good!

[orbeckst]

Note to myself: this is the log message for the squash-merge

• fix treatment of use of SI should be transparent #147
• use exclusive options --SI and --no-SI in the mdpow-pow, mdpow-solvationenergy and mdpow-pcw
• use logging to output use of SI
• replaced optarg with argparse
• fix docs for --plot-file