UnityGLTF icon indicating copy to clipboard operation
UnityGLTF copied to clipboard

Add "ShouldNodeExport" hook

Open andybak opened this issue 1 year ago • 1 comments

Example usage:

    public override bool ShouldNodeExport(GLTFSceneExporter exporter, GLTFRoot gltfRoot, Transform transform)
    {
        Type[] excludedTypes =
        {
            typeof(FooBar),
            typeof(BarBaz)
        };
        bool hasExcludedComponent = excludedTypes.Any(t => transform.GetComponent(t) != null);
        return !hasExcludedComponent;
    }

andybak avatar Aug 26 '24 17:08 andybak

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Aug 26 '24 17:08 CLAassistant

I've just hit a need for a similar method for Import. Does that belong in the same PR?

Is there anything missing that would help move this PR forward? We're a branch including this PR in production now.

andybak avatar Feb 09 '25 12:02 andybak

We're looking at the PR again and looks like it will be included with the next version :) We did add a big warning to the Readme that using this hook requires understanding what can go wrong because it's rather easy to shoot yourself in the foot (e.g. skipping export of a few bones in a hierarchy).

hybridherbst avatar Mar 07 '25 13:03 hybridherbst

Would be great if you take a look at that then and yes, feel free to make a new PR for import functionality!

hybridherbst avatar Mar 07 '25 13:03 hybridherbst

This is now merged and released in 2.15 🎉 (not sure why GitHub doesn't notice that in this PR)

hybridherbst avatar Mar 10 '25 11:03 hybridherbst