-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
dag: Add --mermaid
and --md
option.
#7383
Conversation
0d8ff38
to
f8b4d4c
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.
π₯
We could extend this with
flowchart TD
node1[data/data.xml.dvc]
node2[evaluate]
node3[featurize]
node4[prepare]
node5[train]
node6[data2.dvc]
subgraph P1
node6
end
subgraph P2
node1-->node4
node3-->node2
node3-->node5
node4-->node3
node5-->node2
end
|
That was a doubt I had because I couldn't find an example test. When it's possible for |
--mermaid
option.--mermaid
and --md
option.
This is now inconsistent with other |
I see your point. However, the way I understand it is that On the one hand, the other On the other, this command would generate a I see inconsistency in the fact that this is the first |
Definitely prefer "self-contained markdown component with backticks"
|
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
@daavoo as @0x2b3bfa0 said, maybe |
|
What's the status of this PR? |
This is already the current status |
β I have followed the Contributing to DVC checklist.
π If this PR requires documentation updates, I have created a separate PR (or issue, at least) in dvc.org and linked it here.
Now that
mermaid
can be embedded in github md:dvc dag --md >> dag.md cml send-comment dag.md