mosdef-hub / foyer

A package for atom-typing as well as applying and disseminating forcefields
https://foyer.mosdef.org
MIT License
117 stars 76 forks source link

Drop Python 3.7 #513

Closed mattwthompson closed 1 year ago

mattwthompson commented 1 year ago

PR Summary:

You might want to consider dropping Python 3.7 as it's out of date with much of the rest of the ecosystem. I also took the liberty of ditching a few files that probably aren't used anymore.

PR Checklist


daico007 commented 1 year ago

I agree with dropping python 3.7, I think we bumped into that issue not long ago with unyt dropping 3.7 in their latest release. Currently, it looks like the test is failing on py3.10 build but I am not sure what's causing that still.

mattwthompson commented 1 year ago

Currently, it looks like the test is failing on py3.10 build but I am not sure what's causing that still.

Micromamba complained about not finding an OpenMM build on Windows. That sounds plausibly a source of confusion I don't want to deal with here, so I've just dropped it.

codecov[bot] commented 1 year ago

Codecov Report

Merging #513 (7735fec) into main (25a5fac) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #513   +/-   ##
=======================================
  Coverage   70.84%   70.84%           
=======================================
  Files          16       16           
  Lines        1650     1650           
=======================================
  Hits         1169     1169           
  Misses        481      481