This area had significant changes in the releases past 3.6; it doesn't seem particularly feasible to backport a fix unfortunately.
Closing this issue since it's resolved in more recent versions…
Commit was fixed with 80092fe1e0948f019d5fceb822d8073ed22025b6
I'm closing this issue since we no longer have any more 3.3 LTS releases planned.
Makes more sense that we would return false here instead of true if we're building without opensubdiv
Nit: If we're changing the name from EvaluatorImpl -> Evaluator, I think renaming the file is probably worth doing too. Could wait for a follow up cleanup though
Lowering the severity of this issue since the crash was based on the assets being in a weird state when setting the repo up (mentioned in the linked PR)
Updated the title, this affects more than just masking, it happens for face sets and general positions too. I believe the issue is that we are likely filtering the PBVH nodes to update based on…