Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added regression tests for hydrogenbondautocorrel #917

Merged
merged 4 commits into from
Aug 1, 2016

Conversation

richardjgowers
Copy link
Member

Fixes #

Changes made in this Pull Request:

PR Checklist

  • Tests?
  • Docs?
  • CHANGELOG updated?
  • Issue raised/referenced?

@coveralls
Copy link

coveralls commented Jul 28, 2016

Coverage Status

Coverage increased (+0.9%) to 84.04% when pulling c464960 on hbondautocorreltests into e5c98de on develop.

@orbeckst
Copy link
Member

Ready to merge or do you want to add more? +0.9% is good!

class TestHydrogenBondAutocorrel(object):
def setUp(self):
u = self.u = mda.Universe(TRZ_psf, TRZ)
self.H = u.atoms.Hn
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't rely on the quick selectors and rather do explicit select_atoms.

@orbeckst
Copy link
Member

For some reason, intermittent bonds are not tested, even though you have tests.

@orbeckst orbeckst self-assigned this Jul 28, 2016
@richardjgowers
Copy link
Member Author

Ok this can merge once it passes

@coveralls
Copy link

coveralls commented Jul 29, 2016

Coverage Status

Coverage increased (+1.01%) to 84.14% when pulling 70f7ca8 on hbondautocorreltests into e5c98de on develop.

@orbeckst
Copy link
Member

One test failed. Possibly decrease requested comparison accuracy.

…is into hbondautocorreltests

Conflicts:

testsuite/MDAnalysisTests/analysis/test_hydrogenbondautocorrel.py
@richardjgowers
Copy link
Member Author

Ok I redid the test (and was less lazy about it), should pass now.

@coveralls
Copy link

coveralls commented Aug 1, 2016

Coverage Status

Coverage increased (+1.02%) to 84.141% when pulling 72ceff9 on hbondautocorreltests into e5c98de on develop.

@orbeckst orbeckst added this to the 0.16.0 milestone Aug 1, 2016
@orbeckst orbeckst merged commit 286e327 into develop Aug 1, 2016
@orbeckst orbeckst deleted the hbondautocorreltests branch August 1, 2016 17:33
jdetle pushed a commit to jdetle/mdanalysis that referenced this pull request Aug 14, 2016
abiedermann pushed a commit to abiedermann/mdanalysis that referenced this pull request Oct 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants