Page MenuHome

Fix T95458: Line art ignores curve objects with no evaluated mesh
ClosedPublic

Authored by Hans Goudey (HooglyBoogly) on Feb 7 2022, 11:03 PM.

Details

Summary

Some curve objects don't have an evaluated mesh at all, but line art
currently assumes that all curve objects have one before converting
it to a mesh internally. Fix this by checking if the curve object has an
evaluated mesh before skipping it.

Diff Detail

Repository
rB Blender

Event Timeline

Oh I didn't know this is a problem. I guess it's probably also caused by curve displist rewrite stuff?

Code looks good.

This revision is now accepted and ready to land.Feb 8 2022, 3:42 AM
Hans Goudey (HooglyBoogly) planned changes to this revision.Feb 8 2022, 4:36 PM

Actually, I think this will end up ignoring the "allow duplicates" option for any instance with an evaluated mesh.
I don't really think that option should exist in the first place, but I can think of a solution that will solve it and provide some other benefits too.

Huh alright... The main purpose of it was to ignore particles to speed up calculation. I think there needs to be a more elegant solution as well ๐Ÿ˜…

Hans Goudey (HooglyBoogly) requested review of this revision.Feb 10 2022, 6:31 PM

Actually, I think this change is fine and solved the given problem, my concern above is incorrect.

The remaining problem is that evalauted from non-mesh objects or evaluated curves from non-curve objects, etc. will be ignored if "Allow Duplicates" is off. That's a different problem though.

Huh alright... The main purpose of it was to ignore particles to speed up calculation.

In that case the particles could be split to a separate object or collection I think. But yeah, it's more up to the grease pencil team whether this option should exist or not.

This revision is now accepted and ready to land.Feb 10 2022, 6:31 PM