Build interactive pipeline visualization editor #437
Conversation
Co-authored-by: ss.shankar505 <[email protected]>
|
Cursor Agent can help with this pull request. Just |
This commit introduces a visual editor for configuring pipelines, allowing users to manipulate blocks, properties, and stacks through a user-friendly interface. The JSON editor remains available as an alternative. Co-authored-by: ss.shankar505 <[email protected]>
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on November 6
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| ? "border-destructive" | ||
| : "" | ||
| }`} | ||
| value={String( |
There was a problem hiding this comment.
Bug: Validation Fails for Zero Dimensions
The validation styling for width and height input fields, affecting both main blocks and stack sub-blocks, doesn't appear when the value is 0. This happens because the && operator short-circuits when 0 is evaluated as falsy, preventing the <= 0 check from running.
Additional Locations (3)
| const [moved] = srcStack.operators.splice(source.index, 1); | ||
| const dstStack = next[dpIdx].operators[dbIdx] as StackOp; | ||
| dstStack.operators.splice(destination.index, 0, moved); | ||
| setConfig({ pipelines: next }); |
There was a problem hiding this comment.
Bug: State Mutation Causes React UI Inconsistency
The onDragEnd handler directly mutates operators arrays within pipelines and stacks. The shallow copy of pipelines doesn't prevent modification of nested arrays, which can lead to React missing state updates and inconsistent UI.
This PR adds a visualization tool for us to generate figures for our papers. Example screenshot: