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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(workflow): add dot output on workflow visualize #3032
Conversation
- Fixes also a bug when providing wrong columns fix #2376
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, this looks a lot simpler than the pygraphviz approach!
.. code-block:: console | ||
|
||
$ renku workflow visualize --format dot <path> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe write a sentence that this would print dot format, which could be accepted by the dot
command
tests/cli/test_workflow.py
Outdated
'"Y" -> "bash -c \\"cat X Y | tee R S\\"";' in result.output | ||
'"X" -> "bash -c \\"cat X Y | tee R S\\"";' in result.output | ||
'"bash -c \\"cat X Y | tee R S\\"" -> "R";' in result.output | ||
'"bash -c \\"cat X Y | tee R S\\"" -> "S";' in result.output | ||
4 == result.output.count('"bash -c \\"cat X Y | tee R S\\"') | ||
|
||
1 == result.output.count('"echo other > H" -> "H"') | ||
1 == result.output.count('-> "H"') | ||
0 == result.output.count('"H" -->') | ||
1 == result.output.count('"H"') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of these statements need an assert
in front of them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ouch! 馃う
1def087
to
531869c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
- Fixes also a bug when providing the wrong columns fix #2376 Co-authored-by: Ralf Grubenmann <ralf.grubenmann@sdsc.ethz.ch>
Description
Add a
--format
option on therenku workflow visualize
command. The output can now be eitherconsole
ordot
.Maybe instead of
-i
would it make sense to add an extra format? Something likeconsole
,interactive
,dot
?This also fixes an error when providing non-existing column names
E.G.
renku workflow visualize <path> --columns aaa
was triggeringNow it prints
fix #2376