setzer22 / blackjack

A procedural, node-based modelling tool, made in rust 🦀
Mozilla Public License 2.0
1.42k stars 64 forks source link

Array inputs #55

Open novhack opened 2 years ago

novhack commented 2 years ago

Some nodes like "Merge meshes" would really benefit from having just one input array (in this case an array of meshes). Blender geometry nodes have this feature and indicate it with a "pill" slot. image

setzer22 commented 2 years ago

This is planned, but is currently blocked on this PR of egui_node_graph https://github.com/setzer22/egui_node_graph/pull/30 which is currently being reworked by @mxgrey.

Zireael07 commented 2 years ago

I see this PR also blocks/makes more difficult the idea of a loop node (that would allow executing several nodes X times)

mxgrey commented 2 years ago

I'm doing a rework to achieve multi-input/output that's using a totally separate approach from the PR that was linked. See the last comment in the PR thread.

I'll be making sure the version that I'm working on is compatible with looped graphs, because that's an important requirement for my use cases.

setzer22 commented 2 years ago

Yes, I should've clarified :sweat_smile:, the code for original linked PR went stale, but the ongoing discussion in the thread is still relevant :+1: