Skip to content
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

Node tree workflow optimization. Freezing functionality? #4322

Open
Durman opened this issue Sep 2, 2021 · 0 comments · May be fixed by #4738
Open

Node tree workflow optimization. Freezing functionality? #4322

Durman opened this issue Sep 2, 2021 · 0 comments · May be fixed by #4738
Labels
Proposal 💡 Would be nice to have

Comments

@Durman
Copy link
Collaborator

Durman commented Sep 2, 2021

Problem statement

There is no sense to evaluate nodes if their work can't be detected by user. For example on the next picture evaluation of first group of nodes can be skipped because it effects nothing. The second group, on contrary, should be evaluated because they are connected to output node.
2021-09-02_11-48-53
This example looks artificial because nobody will create net of nodes without output node on the end. Therefore there is second part of the proposal.

There is another special part of nodes which should be always evaluated because they change data outside of a tree. They can be called output nodes. They can be detected by extra button on their interface responsible for their activity state.

2021-09-02_11-58-59

So when such button is disable the node can be considered as frozen. There is no sense update normal nodes before frozen output node. If output node has output sockets they should keep it's last state so next could use it for their work I think.

Use case of this feature is when you have some basic shape which you modify to add extra details it's possible to dissable the branch responsible for adding details to this shape and modify it's basic properties witout performance penalty. In the picture below it will be possible to change Box node properties and only temporal viewer will be reevaluated. Switching on Mesh viewer should re-update all previous nodes.
2021-09-02_12-08-08

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Proposal 💡 Would be nice to have
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant