search

peopledoc / mlvtools

Public repository for versioning machine learning data
Other
42 stars 7 forks source link

DC-80 Export DVC Pipeline #22

Closed sbracaloni closed 6 years ago

sbracaloni commented 6 years ago

Export the ordered list of commands needed to complete a given pipeline step. Sequential execution. Local pipeline step only.

mgu commented 6 years ago

I don't understand how you split your commits (1 commit for the code then 1 for the tests). (I would have made a first commit with only the export code (w/ tests) then another with the command)

sbracaloni commented 6 years ago

@mgu Thanks for the review. Usually I do not split test and code in commit. But It started to become a huge commit and I wanted to make it easier for reviewers. Your proposal is better than mine. Next time I will do that.