Make sure the include is in all updated node files, unity builds might hide missing includes.
It has not been formally agreed upon yet. I'm just pushing for it a bit. Also a few things have changed since the original implementation. E.g. we can properly support panels in the modifier now.
ASAN Report
==182792==ERROR: AddressSanitizer: heap-use-after-free on address 0x7f11d9ac5800 at pc 0x000003062d22 bp 0x7ffcff898870 sp 0x7ffcff898868
READ of…I can reproduce it. It crashes earlier already in a debug build.
Backtrace
__pthread_kill_implementation@0x00007fffe64ae834 (Unknown Source:0)
raise@0x0000…If not filtering by type, maybe we can modify
DEG_OBJECT_ITER_BEGINinto using object_dependencies list as a reference and only return objects that has the original id of those in that list?…
Haven't looked at the code in too much detail yet. It seems the behavior is still different compared to legacy curves.
Looks good to me beside what is mentioned already.
Any specific reason for this very large grain size? Multi-threading should be worth it at way lower grain sizes already, especially since the loop body looks fairly costly with a bvh tree lookup.
The less hacky way would be to follow what physics collections are doing: gather all dependencies during DEG construction, and storing them somewhere for a later use. This way evaluation would…
Since those objects can be distinguished by object types.
I don't really think this is possible. Excluding grease pencil objects here is quite wrong, because the evaluated data of grease…