search

breadboard-ai / breadboard

A library for prototyping generative AI applications.
Apache License 2.0
160 stars 22 forks source link

"Hide Advanced Ports on Nodes" impacts node disambiguation #1733

Open donsbach opened 4 months ago

donsbach commented 4 months ago

When connecting collapsed nodes, if the "Hide Advanced Ports on Nodes" is deselected, the disambiguation shows extra options:

image

It should probably work the same wether this is selected or deselected.

paullewis commented 4 months ago

I think filtering based on type will help here, but if there's a valid port to connect from/to we should probably always show it? Wdyt, @dglazkov?

donsbach commented 4 months ago

Filter by type will help a lot, but in this case where there are multiple types on each side, it wouldn't resolve the problem. It seems like Nodes default ports should be honored wether advanced ports are hidden or not.