Remove settings remove_com from OpenMMSystemGeneratorFFSettings (added to openfe IntegratorSettings)

[hannahbaumann]

No description provided.

[hannahbaumann]

As discussed here (OpenFreeEnergy/openfe#665), moving the remove_com setting to IntegratorSettings.

[pep8speaks]

Hello @hannahbaumann! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file gufe/tests/test_transformation.py:

Line 34:80: E501 line too long (396 > 79 characters)
Line 131:80: E501 line too long (435 > 79 characters)

Comment last updated at 2024-01-26 08:55:55 UTC

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (51e6812) 99.22% compared to head (c3207e0) 99.22%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #275   +/-   ##
=======================================
  Coverage   99.22%   99.22%           
=======================================
  Files          36       36           
  Lines        1938     1938           
=======================================
  Hits         1923     1923           
  Misses         15       15           

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

[richardjgowers]

@hannahbaumann FYI, this key change will probably happen in openfe once this is merged (as the Settings there are now different than before)