Update SMIRNOFF/OpenFF docs/examples #305
Conversation
| # Consider upgrading to 2.1.0, the recommended small moleucle force field for general use | ||
| self._reference_forcefield = 'openff_unconstrained-2.0.0' | ||
| self._charge_method = 'from-molecule' |
There was a problem hiding this comment.
I noticed this in EspalomaTemplateGenerator but didn't want to make the change without better understanding what would happen
There was a problem hiding this comment.
I believe the LJ parameters are unchanged between 2.0.0 and 2.1.0, so this should have no effect. But thanks for adding the comment for us to reevaluate later!
There was a problem hiding this comment.
I think that's right - it's a Pavlovian response of mine 🙂
Codecov ReportAll modified lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #305 +/- ##
=======================================
Coverage 68.03% 68.03%
=======================================
Files 5 5
Lines 879 879
=======================================
Hits 598 598
Misses 281 281
☔ View full report in Codecov by Sentry. |
| # Consider upgrading to 2.1.0, the recommended small moleucle force field for general use | ||
| self._reference_forcefield = 'openff_unconstrained-2.0.0' | ||
| self._charge_method = 'from-molecule' |
There was a problem hiding this comment.
I believe the LJ parameters are unchanged between 2.0.0 and 2.1.0, so this should have no effect. But thanks for adding the comment for us to reevaluate later!
I noticed that the README currently recommends an old (2020?) Parsley force field. I updated the default behavior of
SMIRNOFFTemplateGeneratorto use 2.1.0, our current recommended force field for general use. That should be the only behavior change introduced here.I also took a dive through the examples in the README and many of the examples in the source code and updated them and/or made sure they ran. I don't think there are substantive or wacky changes there.