[Question] MESH_PRIMITIVE_TOO_FEW_TEXCOORDS about sharpgltf

[vpenades]

Could it be possible to share one of these models?

Without seeing the models I suspect the issue is happening because the materials require more texture uv channels than what the mesh primitives provide

[mhwlng]

Try this file

https://habufa-res.cloudinary.com/image/upload/v1691148553/bad.glb

with this viewer

https://gltf-viewer.donmccurdy.com/

[mhwlng]

f.y.i. I edited the first post, because I initially misinterpreted the problem:

It seems that to fix the problem, you need to simply REMOVE "texCoord": 1 from the materials\occlusionTexture in the json

because there is no TEXCOORD_1 defined in any of the mesh primitives

[vpenades]

Well... the model is malformed, and if the library is unable to detect that it is malformed then there's a bug in the validator.

Another thing is if the library should fix it or decide whether the model is fixable.

In this particular case, most probably the material's occlusion requires its own UV channel, because that's how occlusion maps work, so fixing it by using the UVs of channel 0 would cause the occlusion maps to be rendered incorrecly... the only solution, as you mentioned, is to completely remove the occlusion from the materials.

[mhwlng]

I don't know if this is the best way to do it, but this is how I fixed the .glb file:

for (var i = 0; i < rootObject.LogicalMaterials.Count; i++)
{
  var material = rootObject.LogicalMaterials[i];
  
  var hasOcclusion = material.Channels
  .FirstOrDefault(o => o.Key == "Occlusion" && o.TextureCoordinate > 0).Key != null;
  
  if (hasOcclusion)
  {
    var materialBuilder = material.ToMaterialBuilder();
    materialBuilder.RemoveChannel(KnownChannel.Occlusion);
    materialBuilder.Name = "Repair_" + material.Name;
    
    var newMaterial = rootObject.CreateMaterial(materialBuilder);
    
    foreach (var mesh in rootObject.LogicalMeshes)
    {
      foreach (var meshPrimitive in mesh.Primitives)
      {
        if (meshPrimitive.Material == material)
        {
          meshPrimitive.Material = newMaterial;
        }
      }
    }
  }
}

rootObject.SaveGLB("test.glb", new WriteSettings()
{
  Validation = ValidationMode.TryFix
}
);

Of course, There are now one or more unused materials in the GLB. So, the new GLB file is bigger.

[vpenades]

Yes, right now that would be the way to do it.