Curve Viewer Draw & Surface Viewer Draw nodes. #4652
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
portnov
force-pushed
the
curve_surface_viewer_nodes
branch
from
89aae57
to
a1d710a
Compare
|
It would be logical. Also it seems that Ctrl + LMB could connect the new viewer to curve and surface sockets. |
from right click menu.
1) when loading (default) preset, node name can be changed, so the reference to the node instance is no longer valid. So when loading a peset, we now return the instance of newly created node. 2) When saving a preset from Shift-P menu, is_node_preset flag was not set, so the preset was saved in incorrect format and could not be loaded as a node preset. 3) Use default presets when creating temporal viewers.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Shorthand nodes for viewing curves and surfaces in 3D View.
In my experience, I tend to use 4 nodes to inspect the curve I generated: Evaluate Curve + Viewer Draw, and Deconstruct Curve + Viewer Draw. Similarly with surfaces.
These nodes allow to replace four nodes with one.
For NURBS, the nodes can show control polygon / control net; "Viewer Draw Curve" node can also show "node points" (Greville points) of the curve.
For non-NURBS, the nodes just do not try to show these NURBS-specific things.
The nodes also have "bake" capability.
Preflight checklist
Put an x letter in each brackets when you're done this item: