items_mutable_span
to items_span_for_write
.
I opted for items_span_for_write
now, if that's ok for you.
We already have NodeSimulationItem *items
for the array, so Span<NodeSimulationItem> items()
is a conflict, otherwise items
+ items_for_write
would work.
Didn't realize we had a dedicated operator py file for geonodes, will move it there.
For the color thing, I think I still prefer clarity/simplicity in the API and pushing the complexity to an RNA helper function.
Problem is the std_node_socket_colors
list is currently part…
Renamed the functions and added comments for clarity. Also the copy_simulation_state_to_output_param
function is now shared by input and output node, they both need to do this: input node when…
Was just an ad-hoc name: it's copying from the simulation state into the output parameter for the next iteration. I'll add some comments.
Implicit cast to bool does not work with enum class
unfortunately (suggested by Jacques above). Functional cast should be fine though.
Also note i've added C++ spans to the NodeGeometrySimulationOutput
now to make this a bit nicer.