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

cli: add pipeline show --tree #1781

Merged
merged 3 commits into from
Mar 26, 2019
Merged

cli: add pipeline show --tree #1781

merged 3 commits into from
Mar 26, 2019

Conversation

puhoshville
Copy link
Contributor

Add option to unfold dependencies into a tree.

Fixes #1038

Add option to unfold dependencies into a tree.

Fixes #1038
Copy link
Contributor

@efiop efiop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great patch! 🎉 A few comments down below. Also, would you mind showing us an example of dvc pipeline show --tree output?

requirements.txt Show resolved Hide resolved
dvc/command/pipeline.py Outdated Show resolved Hide resolved
@puhoshville
Copy link
Contributor Author

Great patch! 🎉 A few comments down below. Also, would you mind showing us an example of dvc pipeline show --tree output?

Example of dvc pipeline show --tree output
For following graph
dvc pipeline show --ascii e.file.dvc

+------------+
| a.file.dvc |
+------------+
       *
       *
       *
+------------+
| b.file.dvc |
+------------+
       *
       *
       *
+------------+              +------------+
| c.file.dvc |              | d.file.dvc |
+------------+              +------------+
             ***           ***
                **       **
                  **   **
              +------------+
              | e.file.dvc |
              +------------+

we got following tree:
dvc pipeline show --tree e.file.dvc

e.file.dvc
├── c.file.dvc
│   └── b.file.dvc
│       └── a.file.dvc
└── d.file.dvc

@efiop
Copy link
Contributor

efiop commented Mar 26, 2019

@puhoshville Thanks! :) Could you also address this one #1781 (comment) please? You just need to raise DvcException instead of ValueError.

@efiop efiop merged commit 35218a7 into iterative:master Mar 26, 2019
@efiop
Copy link
Contributor

efiop commented Mar 26, 2019

@puhoshville Thank you so much! 🎉

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.

2 participants