Migrate GOMC and CHARMM writer

[daico007]

PR Summary:

Now that https://github.com/GOMC-WSU/MoSDeF-GOMC is in decent shape. I think it's time to migrate the CHARMM and GOMC writers that initially added here by @bc118.

PR Checklist

---

• Includes appropriate unit test(s)
• Appropriate docstring(s) are added/updated
• Code is (approximately) PEP8 compliant
• Issue(s) raised/addressed?

[lgtm-com]

This pull request fixes 11 alerts when merging d535ea4 into 846e18e - view on LGTM.com

fixed alerts:

• 8 for Unused local variable
• 2 for Except block handles 'BaseException'
• 1 for Unreachable code

---

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[bc118]

This is fine with me as long as it passes all the tests.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.69%. Comparing base (671572f) to head (d1031b2).
Report is 142 commits behind head on main.

Files with missing lines	Patch %	Lines
mbuild/utils/io.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1064      +/-   ##
==========================================
+ Coverage   90.56%   90.69%   +0.13%     
==========================================
  Files          60       57       -3     
  Lines        8406     5397    -3009     
==========================================
- Hits         7613     4895    -2718     
+ Misses        793      502     -291     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[lgtm-com]

This pull request fixes 11 alerts when merging 52c3988 into 671572f - view on LGTM.com

fixed alerts:

• 8 for Unused local variable
• 2 for Except block handles 'BaseException'
• 1 for Unreachable code

---

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[daico007]

Note to self: need to refer to this https://github.com/mosdef-hub/mbuild/pull/1029/files for other changes that could be removed.

[lgtm-com]

This pull request fixes 11 alerts when merging 9fcfb71 into 671572f - view on LGTM.com

fixed alerts:

• 8 for Unused local variable
• 2 for Except block handles 'BaseException'
• 1 for Unreachable code

---

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[lgtm-com]

This pull request fixes 12 alerts when merging d1031b2 into 671572f - view on LGTM.com

fixed alerts:

• 9 for Unused local variable
• 2 for Except block handles 'BaseException'
• 1 for Unreachable code

---

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.