Port Snapping doesn't take InputParamKind::ConstantOnly into account about egui_node_graph HOT 4 OPEN

Addendum: This affects any output port, no matter the type as long as at least one input in the graph has the InputParamKind::ConstantOnly. Changing the InputParamKind to allow connections makes the crash disappear.

As an aside, I have noticed that without the crash the UI will snap to ports that are not the same color even though it won't actually make a connection. I don't know if this behavior should be standard or if it might confuse users who may think the snap indicates the port is valid.

from egui_node_graph.

However, thinking about it again, it is a difficult problem because it would be inconsistent to allow different colors to be connected.

@kkngsm The current system will get reworked once #30 is merged (might take a while) to allow specifying compatibility between different kinds of ports using a custom function. Once we have that function, we'll be able to use it to indicate more complex logic for when two ports are compatible. For the time being, the semantics for this library is that a connection cannot be made between ports of different type.

from egui_node_graph.

The crashing issue has been corrected. I will PR it later.

As an aside, I have noticed that without the crash the UI will snap to ports that are not the same color even though it won't actually make a connection. I don't know if this behavior should be standard or if it might confuse users who may think the snap indicates the port is valid.

Indeed, I had not thought of this.
However, thinking about it again, it is a difficult problem because it would be inconsistent to allow different colors to be connected.

from egui_node_graph.

This was wrongly closed by GitHub when I merged #59, there is still the pending issue of only snapping to compatible ports with the same color.

from egui_node_graph.