ref: main exp workflow (run,show,diff,apply) #2200
Conversation
jorgeorpinel
changed the title
| "source": "exp/index.md", | ||
| "children": [ | ||
| { | ||
| "label": "exp run", | ||
| "slug": "run" | ||
| }, |
There was a problem hiding this comment.
Hiii @rogermparent another quick callout... Would it be possible to automatically label subcommands inside children as parent children to avoid all these {} structures (here and in other subcommand groups). Ty
There was a problem hiding this comment.
Something like replacing { "label": "exp run", "slug": "run" } with the string "run" having the same effect? That's a good idea, and totally doable!
There was a problem hiding this comment.
Great! I was just thinking to replace the whole Yep, exactly (:{} block with "run" in this case
There was a problem hiding this comment.
Any hope we can get that done for this PR @rogermparent? No worries if not but let's make make an issue then? Thanks
jorgeorpinel
temporarily deployed
to
dvc-org-ref-exp-main-rbzbwyywk
jorgeorpinel
temporarily deployed
to
dvc-org-ref-exp-main-rbzbwyywk
jorgeorpinel
temporarily deployed
to
dvc-org-ref-exp-main-rbzbwyywk
jorgeorpinel
temporarily deployed
to
dvc-org-ref-exp-main-rbzbwyywk
jorgeorpinel
had a problem deploying
to
dvc-org-ref-exp-main-rbzbwyywk
and reorder exp run options
jorgeorpinel
temporarily deployed
to
dvc-org-ref-exp-main-rbzbwyywk
jorgeorpinel
force-pushed
the
ref/exp/main
branch
2 times, most recently
from
59412a9
to
a16d83e
Compare
|
LGTM @shcheklein @dberenbaum @pmrowla ? Examples will be coming in a separate PR (this one is large enough). The remaining commands also have their own PR (#2243). |
jorgeorpinel
force-pushed
the
ref/exp/main
branch
from
78b376e
to
0fb378c
Compare
|
OK guys I'm going to merge this. There will be more iterations in case anyone has more comments. Thanks! |
exp run/show/diff/applyDescs and OptionsI left out the examples in this PR b/c it's getting too big.