search

MDAnalysis / panedr

Reads Gromacs EDR file to populate a pandas dataframe
GNU Lesser General Public License v2.1
31 stars 7 forks source link

Add coverage #29

Closed IAlibay closed 2 years ago

IAlibay commented 2 years ago

Adding a codecov action would be nice for keeping track of where things are at tests wise.

BFedder commented 2 years ago

So there is a codecov action defined in the .yaml file #32 will add, but I am not sure if this will work out of the box or need further adaptation from GridDataFormats to panedr

orbeckst commented 2 years ago

Coverage uploads https://app.codecov.io/gh/MDAnalysis/panedr so we really only need the badge #35

orbeckst commented 2 years ago

82% coverage is not too shabby to start with...

orbeckst commented 2 years ago

But it's worthwhile taking a look at https://app.codecov.io/gh/MDAnalysis/panedr/blob/master/panedr/panedr.py and seeing if there's something that really ought to be tested. One can always raise specific issues for certain tests.