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

Nodes to be variables #1885

Open
knowledgenude opened this issue Sep 23, 2020 · 3 comments
Open

Nodes to be variables #1885

knowledgenude opened this issue Sep 23, 2020 · 3 comments
Labels
good first issue Relatively easy-to-solve issues for first-time contributors logic nodes This issue is about logic nodes

Comments

@knowledgenude
Copy link
Collaborator

knowledgenude commented Sep 23, 2020

  • Collection
  • Transform
  • Scene
@knowledgenude knowledgenude added the logic nodes This issue is about logic nodes label Sep 23, 2020
@MoritzBrueckner
Copy link
Collaborator

Nope, they aren't but I agree that they probably should be used as variables.

Regarding the inputs: why shouldn't they have inputs? Actually, this is much easier to use than the Set Variable reverse thingy node.

@knowledgenude
Copy link
Collaborator Author

I mean if we put a text box like we have in On Event, and the same style for variables like object (i am not sure if there is a way to do that for objects).

For example, in the Collection node it have a way to put a "default" value but not a input properly.

I think this would leave the things more clean, but i am not sure about the usability of these inputs.

@MoritzBrueckner
Copy link
Collaborator

I mean if we put a text box like we have in On Event

You can have both an input socket and a textbox/property search field :)

Variable nodes act as a "storage" where you can save and read values from. The property search field is only the initial, default value. I think we should actually add inputs to variable nodes as well and extend the entire variable functionality. Of course this still needs to be documented(!) and it actually might be a bit confusion to have both inputs and the Set Variable node.

@knowledgenude knowledgenude changed the title This nodes are not variables? Nodes to be variables Sep 25, 2020
@MoritzBrueckner MoritzBrueckner added the good first issue Relatively easy-to-solve issues for first-time contributors label Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Relatively easy-to-solve issues for first-time contributors logic nodes This issue is about logic nodes
Projects
None yet
Development

No branches or pull requests

3 participants
@MoritzBrueckner @knowledgenude and others