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

Pr 2036 #5

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Pr 2036 #5

wants to merge 3 commits into from

Conversation

jbarnoud
Copy link
Owner

Fixes #

Changes made in this Pull Request:

PR Checklist

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

orbeckst and others added 3 commits August 9, 2018 17:11
- test functions that require seaborn (skip if cannot be imported)
- updated warning for plot_nearest_neighbors() to use run_pairs_analysis(neighbors=True)
@codecov-io
Copy link

codecov-io commented Aug 10, 2018

Codecov Report

Merging #5 into develop will increase coverage by 0.29%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop       #5      +/-   ##
===========================================
+ Coverage    88.59%   88.89%   +0.29%     
===========================================
  Files          143      143              
  Lines        17361    17361              
  Branches      2658     2658              
===========================================
+ Hits         15381    15433      +52     
+ Misses        1379     1321      -58     
- Partials       601      607       +6
Impacted Files Coverage Δ
package/MDAnalysis/analysis/psa.py 80.61% <ø> (+9.98%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10fb62d...e3eee85. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants