search

MDAnalysis / pmda

Parallel algorithms for MDAnalysis
https://www.mdanalysis.org/pmda/
Other
31 stars 22 forks source link

add new timers for "map" process and time for blocks to start working #82

Closed VOD555 closed 5 years ago

VOD555 commented 5 years ago

Changes made in this Pull Request:

PR Checklist

orbeckst commented 5 years ago

Also CHANGELOG & docs!

VOD555 commented 5 years ago

@orbeckst All the changes made in the PR have passed the tests. Please check whether there is something need to be modified.

orbeckst commented 5 years ago

Please fix PEP8 ...

orbeckst commented 5 years ago

Also issues in LeafLet that are related to the changes.

E       TypeError: __init__() missing 1 required positional argument: 'wait'
orbeckst commented 5 years ago

I fixed #83 and merged master into this branch so now all tests should pass.

codecov[bot] commented 5 years ago

Codecov Report

Merging #82 into master will increase coverage by 0.03%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
+ Coverage   97.14%   97.18%   +0.03%     
==========================================
  Files           8        8              
  Lines         491      497       +6     
  Branches       70       70              
==========================================
+ Hits          477      483       +6     
  Misses          8        8              
  Partials        6        6
Impacted Files Coverage Δ
pmda/parallel.py 98.37% <100%> (+0.08%) :arrow_up:

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 aa1fd21...6e046da. Read the comment docs.