-
Notifications
You must be signed in to change notification settings - Fork 109
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
Sockets do not accept more than one plug #20
Comments
Hi, I will take a look when I have some free time |
This would be a nice feature so long as it could be enabled or disabled by an argument when creating a socket. |
Sorry guys, I really don't have time to look into this. |
I'll need that too, so I'll start working on it. |
Can you do it with a flag to enable/disable directly exposed at attribute creation ? |
Actually I intend to do it with an integer (rather than a flag) that tells the maximum connections count. Default values will be -1 for plugs (infinite connections) and 1 for sockets (only 1 connection), so that it doesn't change the behavior for the current code. Does it sound oK for you ? |
Sounds good ! |
that would be very cool! I use nodz to graph asset hierarchy. My solution is to create multiple plugs for "parent" every time a new connection is added. This makes the graphs ugly but solves the problem. I would love to be able to just connect multiple connections to a socket. Plugs can already to it so why not sockets? |
It was just how i designed it for a specific use case. But it should be fairly simple tu modify. |
a quick and dirty solution is to comment out a few lines in the SlotItem.connect method.
without that sockets are like plugs. |
…lickSignal Added a signal_NodeRightClicked that is emitted when the user right click on a node
Hi,
currently sockets do not accept more than one plug. However, I would like to be able to connect several plugs to a socket (e.g. for routing MIDI messages). I found an if statement in the
accepts
method of theSocketItem
class (line 1833) which returnsFalse
if more than one plug is connected. However, uncommenting the corresponding line does not seem to do the trick. Can you give me a hint which lines of code I have to change?Best,
Hanno
The text was updated successfully, but these errors were encountered: