Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Multiple Connection on node input #31

Closed
muit opened this issue Feb 1, 2016 · 17 comments
Closed

Multiple Connection on node input #31

muit opened this issue Feb 1, 2016 · 17 comments

Comments

@muit
Copy link

muit commented Feb 1, 2016

Hi!
First of all, nice work, this really helps a lot!
So the question/possible feature:
Is it possible to create a connection type that can be connected multiple times with only one input?
like:
A connected to C,
B connected to C

@Seneral
Copy link
Owner

Seneral commented Feb 1, 2016

Not yet without further modifications, but there is currently a discussion on the thread (last page). In short, I'll do this including other customization options when I'm finished with the statemachine support. You may also take a look at the other 'issue', I count them as one as they both adress customization limits of knobs. Priority to change this is high, so don't worry, in about two weeks I hope to have something done:)
As a side note; Why would you wan't to do this? Besides some abstract usage, no use case comes to my mind that could benefit from this...

@Seneral
Copy link
Owner

Seneral commented Feb 1, 2016

Btw I could implement this really fast if you wish. Not too much to do;)

@muit
Copy link
Author

muit commented Feb 1, 2016

Im working on a script/screenplay editor for writers. (Will also publish it with some other tools when I have it working propertly). Thats why I need a node net.

PD: Thanks for your fast answer and work.

@muit
Copy link
Author

muit commented Feb 1, 2016

For now I will create a dynamic input creation :P

@Seneral
Copy link
Owner

Seneral commented Feb 1, 2016

Also a good idea. But for a writer (kind of a dialogue system I suppose?), the upcoming Statemachine behaviour would be preferrable I guess. I have it somewhat working currently, not very fleshed out, but it supports transitioning conditions and time, so I guess I can prepare you a alpha/beta whatever until next weekend. Maybe I'll even have it working fully until then;)

@muit
Copy link
Author

muit commented Feb 1, 2016

Really? Conditions like the unity animation control? Thats cool, and also something I planned to work on. Could I help you someway?

@Seneral
Copy link
Owner

Seneral commented Feb 1, 2016

Yes, this would be features the base transition has. For your specific needs I'll implement a fetching system similar to the ones of Nodes and ConnectionTypes. That will enable you to create transitions dedicated to transition between animations, AI states or the screenplay elements in your case.
I don't know exactly how you could help, sorry. It's a thing to share tasks between contributors and another to work on one task together... I don't know where your interests lie, but the Connections still need some sort of click detection and clipping feature (both pretty easy for lines, not so for beziers). But I appreciate your help and hope you're doing well creating you Screenplay editor:)

@muit
Copy link
Author

muit commented Feb 1, 2016

:P Thanks for your support, I will probably improve the key control system, like move multiple nodes or create connections from outputs to inputs. if I do it i will make PR.

@Seneral
Copy link
Owner

Seneral commented Feb 1, 2016

Sounds great, good luck!

A hint, I used this before in another project: You can get the Editor selection rect style by using GUIStyle boxSelectionStyle = "SelectionRect"
Of course only a style to use for the editor version, nor at runtime:(

@muit
Copy link
Author

muit commented Feb 1, 2016

Cool, thanks. I will do that then.

@nicloay
Copy link

nicloay commented Feb 5, 2016

@Seneral Thanks for support. I faced exact the same problem. I'm working on some kind of dialogue system where I'm triggering different events on the different scene objects. and i also has some loops in logic where i need to go back to previous events after execution.

Could you give me a hint. in case if I'll fix this by myself what is a better, modify existing NodeInput.cs connection property, or derive from nodeKnob and create brand new class with implementation.

thx.

@Seneral
Copy link
Owner

Seneral commented Feb 5, 2016

I recently explored the option of creating multiple input connections, even implemented it, but as a cause all saves got broken... Don't know if that step makes any good...
Regarding your requests, as mentioned before, you needs would be best fulfilled with the WiP state behaviour system I suppose:)

@Seneral
Copy link
Owner

Seneral commented Feb 10, 2016

I've this implemented as of now. But as any use case I could think of (and both of yours) can be resolved using the new state system, and the implementation breaks existing saves, I'm not sure whether to publish such an update. I probably could write a workaround to load old saves, but I'd like to avoid that if possible.
Instead, you're actually able to extend the NodeKnobs way better with other changes I've made, so you can reproduce the behaviour of existing Inputs/Outputs yourself if needed (With restrictions, fE. no direct access through Inputs[] or Outputs[] but through the generic knob collection 'mess' NodeKnobs[]).
What do you think, should I include this change in a new update? The state system will be coming in the same:)

@Seneral Seneral closed this as completed Feb 13, 2016
@vicenterusso
Copy link

Hi, is this feature live in any branch?

@Seneral
Copy link
Owner

Seneral commented Mar 31, 2017

No not yet, been very busy recently:/

@molocai
Copy link

molocai commented Mar 31, 2017

Hello, I'm really interested in this as well, do you have any idea of when it could be live? Or how someone would go about implementing this? Thanks :)

@Seneral
Copy link
Owner

Seneral commented Mar 31, 2017

We have a detailed discussion on this and already an implementation by @JoshAshby but I haven't heard anything about it since:/
I'm currently relatively busy but I expect to get some free time next month:)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants