Passing the graph info to the constructor now, same as get_bake_lazy_function
, and then insert the socket mapping in the constructor. Would be nice to have some kind of correctness check in…
There is a bug in both the Index Switch and Menu Switch nodes here: If you remove all items the set_default_remaining_node_outputs
call will crash, because the lf_index_by_bsocket
for the…
Ok, IndexMask::from_groups
looks like it could work here. I originally swapped the inner and outer loops to be able to devirtualize the input arrays one at a time. The implementation in `Index…
Did IndexMask construction change? I implemented this months ago, and IIRC back then constructing a full index mask for each enum case was basically doing the same thing, looping over the full…
Yes this is a runtime flag. Moving the flag into RuntimeNodeEnumItems
doesn't work though, because when there is a conflict the enum_items
pointer is set to null. The items themselves are…
I removed the menu socket from bake items. It already gets rejected by Bake nodes and simulation zones due to the SimulationItemsAccessor::supports_socket_type
list, so this should be safe. We…
I didn't need to add another update flag after all. I just make sure the enum items of the menu switch are only replaced when the node changes, and then compare the new node tree interface…