Logging fix

[mbkumar]

This PR fixes bugs encountered in importing initialize_logging from simsopt.util.logging reported in #178.

functools.wraps works only for functions, so using SimsoptRequires, which uses functools.wraps internally, on classes won't work as expected. Hence, SimsoptRequires is removed in MHD modules.

[codecov]

Codecov Report

Merging #184 (7e2906d) into master (a25ac26) will increase coverage by 0.00%.
The diff coverage is 66.66%.

❗ Current head 7e2906d differs from pull request most recent head f611e74. Consider uploading reports for the commit f611e74 to get more accurate results

@@           Coverage Diff           @@
##           master     #184   +/-   ##
=======================================
  Coverage   90.51%   90.52%           
=======================================
  Files          57       57           
  Lines        7455     7449    -6     
=======================================
- Hits         6748     6743    -5     
+ Misses        707      706    -1     

Flag	Coverage Δ
unittests	90.52% <66.66%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
src/simsopt/_core/graph_optimizable.py	89.64% <ø> (-0.03%)	⬇️
src/simsopt/mhd/boozer.py	80.85% <ø> (-0.11%)	⬇️
src/simsopt/mhd/spec.py	88.01% <ø> (-0.10%)	⬇️
src/simsopt/mhd/vmec.py	89.20% <ø> (-0.05%)	⬇️
src/simsopt/util/log.py	0.00% <ø> (ø)
src/simsopt/util/mpi_logger.py	0.00% <0.00%> (ø)
src/simsopt/util/mpi.py	91.59% <100.00%> (ø)

---

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 a25ac26...f611e74. Read the comment docs.

[landreman]

If you are removing SimsoptRequires from classes, does it make sense to also remove monty.dev.requires (which also uses functools.wraps, see http://guide.materialsvirtuallab.org/monty/_modules/monty/dev.html#requires) from simsopt.util.mpi.MpiPartition? This is causing issue #159.

[mbkumar]

~~Earlier `monty.dev.requires` was not using `functools.wraps`. In recent versions, they might have changed it. This change obsoletes the need for `SimsoptRequires`.~~ ~~Anyhow, let's not use either of them to decorate classes because functools.wraps is not working for classes.~~ Sorry I forgot why I introduced SimsoptRequires. It was introduced to make sure instancecheck works, by redefining __instancecheck__. I am going to use `SimsoptRequires` in place of `dev.monty.requires` and see if it fixes the docs issue.