Although this avoids the crash by clamping to_index
to a valid value, I believe it still ends up choosing the wrong value for to_index
, which I think means it incorrectly re-orders the custom…
I was asked in chat to confirm this, and indeed I also get this crash. Here's the environment where I tested it: Operating system: macOS-13.6-arm64-arm-64bit 64 Bits Graphics card: Metal API…
@antonheg Yeah, sorry, it turns out I got a bit ahead of myself: I was so excited when GP worked after a factory reset that I completely forgot that the factory reset disabled GPv3 in the…
Actually, I don't think GPv3 is broken on MacOS: I just reset my startup scene to factory settings, and now it seems to be working fine. The various UI elements that weren't showing up in Draw…
Ack, I'm not able to create a test .blend
file for the grease pencil because GPv3 is currently very broken on MacOS: #113601
I get the sense that's not expected to be fixed particularly soon,…
FYI, I've picked this up, and I believe I have it working locally for regular curves. I just need to extend the behavior to grease pencil curves, since the Fill Curve node recently gained support…
A few comments came up in #nodes-physics-module when I asked for reviewers a little while ago, and I had to step away before I got to address them there. Probably better to discuss them here…
@mod_moder If I'm understanding your question correctly, then yes. It seems like it's important that something call DepsgraphNodeBuilder::build_object()
on the relevant cameras to ensure there…
A note/question for the code review: I thought I would need an additional case like this in DepsgraphNodeBuilder::build_nodetree()
to guarantee that the necessary depsgraph nodes would exist for…
@dbsc, I apologize that I completely overlooked the existence of this pull request until I had already completed my own implementation for an Active Camera node. I decided to submit my own pull…
I want to acknowledge that there is another (pre-existing) pull request by someone else for this same issue: !106065
I unfortunately overlooked the existence of that pull request until I had…