Closed flongford closed 5 years ago
Merging #13 into master will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## master #13 +/- ##
=======================================
Coverage 94.55% 94.55%
=======================================
Files 34 34
Lines 826 826
Branches 98 98
=======================================
Hits 781 781
Misses 31 31
Partials 14 14
Impacted Files | Coverage Δ | |
---|---|---|
force_gromacs/pipeline/gromacs_pipeline.py | 100% <100%> (ø) |
: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 d054884...3f2cf19. Read the comment docs.
This PR refactors the
GromacsPipeline
class, in particular:named_steps
property onGromacsPipeline
into a traitsProperty
class, rather than using the@property
decorator.named_steps
is also simplified to remove redundant code_iter
method into a regular python__iter__
method, that no longer returns the index of iteration.bash_script
andrun
methods to account for these changestraits
library.