[doc] Add switch task demo#10247
[doc] Add switch task demo#10247zhongjiajie merged 8 commits intoapache:devfrom huangchenguang123:feature/add_switch_demo
Conversation
QuakeWang
left a comment
There was a problem hiding this comment.
And the screenshots corresponding to the documentation are recommended to be in light mode. Thanks a lot : )
QuakeWang
left a comment
There was a problem hiding this comment.
The provided images can be marked with corresponding tags at key information. In this way, it can be more beneficial to users.
| @@ -1,10 +1,12 @@ | |||
| # Switch | |||
|
|
|||
| Switch是一个条件判断节点，依据[全局变量](../parameter/global.md)的值和用户所编写的表达式判断结果执行对应分支。 | |||
There was a problem hiding this comment.
It is recommended to use the original expression, which is more complete.
| @@ -14,26 +15,33 @@ Drag from the toolbar <img src="/img/switch.png" width="20"/> task node to canv | |||
| - Descriptive information: Describe the function of the node. | |||
There was a problem hiding this comment.
Remove extra indent spaces
|
@QuakeWang Thanks.I had fixed those problems. |
|
I had fixed those problems. |
Co-authored-by: QuakeWang <45645138+QuakeWang@users.noreply.github.com>
|
I find out we still have a discussion in #10247 (comment) and I do not have much experience for switch tasks. Maybe you guys should discuss on it |
What do you think about it?Do you have more suggestions? @QuakeWang |
QuakeWang
left a comment
There was a problem hiding this comment.
This PR sets the corresponding switch branch using global variables. In this way, users can intuitively understand how to use this task.
zhongjiajie
left a comment
There was a problem hiding this comment.
Thanks @huangchenguang123 and @QuakeWang for the review
|
I solve some upstream change in branch dev |
docs/docs/en/guide/task/switch.md
Outdated
| The switch task needs to be connected with the downstream task to configure the relationship before the downstream task can be selected. | ||
|
|
||
| ## Related Task | ||
|  |
There was a problem hiding this comment.
You should use related path instead of absolute path after #10325 merged
There was a problem hiding this comment.
I had change the img path.
|
PTAL @zhongjiajie |
Co-authored-by: QuakeWang <45645138+QuakeWang@users.noreply.github.com> (cherry picked from commit 7e39e0d)
Co-authored-by: QuakeWang <45645138+QuakeWang@users.noreply.github.com>
close #8029