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

Better pyemma integration #19

Merged
merged 23 commits into from
Mar 23, 2017
Merged

Better pyemma integration #19

merged 23 commits into from
Mar 23, 2017

Conversation

gph82
Copy link
Collaborator

@gph82 gph82 commented Mar 16, 2017

No description provided.

@codecov-io
Copy link

codecov-io commented Mar 17, 2017

Codecov Report

Merging #19 into master will increase coverage by 4.9%.
The diff coverage is 92%.

@@            Coverage Diff            @@
##           master      #19     +/-   ##
=========================================
+ Coverage   57.95%   62.86%   +4.9%     
=========================================
  Files          10       11      +1     
  Lines        1301     1489    +188     
  Branches      263      313     +50     
=========================================
+ Hits          754      936    +182     
- Misses        501      502      +1     
- Partials       46       51      +5
Impacted Files Coverage Δ
molpx/tests/test_molPX.py 94.28% <100%> (+8.99%) ⬆️
molpx/bmutils.py 76.96% <78.57%> (+1.73%) ⬆️
molpx/tests/test_bmutils.py 95.57% <90%> (-1.46%) ⬇️
molpx/visualize.py 94.26% <93.58%> (+4.05%) ⬆️
molpx/tests/test_visualize.py 97.59% <97.59%> (ø)

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 ecb2f5c...b99676a. Read the comment docs.

@gph82 gph82 force-pushed the better_pyemma_integration branch from 832630f to 4633230 Compare March 21, 2017 14:33
@gph82
Copy link
Collaborator Author

gph82 commented Mar 22, 2017

@marscher any idea why the coverage tests aren't triggered again on this PR?

@marscher
Copy link
Member

I'd guess, because CI is not yet finished.

@gph82 gph82 force-pushed the better_pyemma_integration branch 3 times, most recently from 462d3c5 to 41f2b40 Compare March 23, 2017 09:22
@gph82 gph82 force-pushed the better_pyemma_integration branch from 41f2b40 to 2d231dc Compare March 23, 2017 09:24
@gph82 gph82 force-pushed the better_pyemma_integration branch 2 times, most recently from 462d3c5 to 2d231dc Compare March 23, 2017 09:28
@gph82 gph82 merged commit b129d76 into master Mar 23, 2017
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