Add cube and light visibility test models for KHR_node_visibility

[aaronfranke]

Same as PR KhronosGroup/glTF-Test-Assets-Interactivity#3, I was told to make a PR to both repos, because this is interactivity-related and interactivity implementers will need this, but also this makes sense outside of interactivity.

---

This model demonstrates the KHR_node_visibility extension and its interaction with KHR_animation_pointer.

• If your implementation shows either of the red cubes, it is not compliant with KHR_node_visibility.

• If your implementation does not have the blue cube hiding and showing every 0.5 seconds, either the animation is not playing, or it does not support using KHR_animation_pointer to animate the KHR_node_visibility visible property.

Animated screenshot of the test model in action:

---

The other model demonstrates the same but with lights.

• If your implementation shows either of the red spotlights, it is not compliant with KHR_node_visibility.

• If your implementation does not have the blue spotlight hiding and showing every 0.5 seconds, either the animation is not playing, or it does not support using KHR_animation_pointer to animate the KHR_node_visibility visible property.

---

Note: These 6 Godot PRs are required to fully fix this test asset in Godot:

• Tracker: [TRACKER] Fixes for KHR_node_visibility godotengine/godot#111618
• GLTF: Support animating node visibility godotengine/godot#111341
• GLTF: Enforce STEP interpolation for integer and boolean animations godotengine/godot#111615
• GLTF: Move accessor and buffer view Dictionary conversion into those classes godotengine/godot#108320
  • GLTF: Move accessor encoding functions to GLTFAccessor godotengine/godot#108853
    • GLTF: Write integer min/max for integer accessors godotengine/godot#111612
    • GLTF: Determine the component type when encoding object model properties godotengine/godot#111613

[lexaknyazev]

Consider not updating LICENSE.md, README.md, Models*.md, and model-index.json files as they should be regenerated by bots anyway.

[aaronfranke]

Fixed those issues.

Consider not updating LICENSE.md, README.md, Models*.md, and model-index.json files as they should be regenerated by bots anyway.

The PR has to include updating these files, because otherwise the CI will complain about them not being updated.

[javagl]

otherwise the CI will complain about them not being updated.

I'm not seeing any failed build here - am I overlooking something?

Only the metadata.json and the README.body.md should be required (at least, I hope so - I didn't add any other files e.g. in #244 ...). At least, that's what's also mentioned in https://github.com/KhronosGroup/glTF-Sample-Assets/blob/main/SubmittingModels.md

[aaronfranke]

@javagl I was getting CI failures when I first opened the PR: https://github.com/KhronosGroup/glTF-Sample-Assets/actions/runs/18479060875/job/52649972324

So I ran all the CI stuff locally to get the asset fully fixed.

I have not actually tested if the CI shows a failure with some of these changes are excluded, but I'm confused as to what the point would be if these changes will be made anyway.

[javagl]

, but I'm confused as to what the point would be if these changes will be made anyway.

The idea is that only the metadata.json and the README.body.md should be part of the PR, and the other files should be auto-generated. This is to ensure that the main README for all assets has the right structure (sections like "Tags"/"Extensions Used"/.../"Legal" etc), and that the license file is present, follows a common format, and refers to valid licenses, and that the directory layout and naming conventions for the sample models are followed.

(The amount of work that people are putting into things like https://github.com/cx20/gltf-test is astonishing - but imagine how much harder that would be if everbody could just check in arbitrary files can call them "sample assets"...)

The build failure (that refers to the state from https://github.com/KhronosGroup/glTF-Sample-Assets/tree/741e2185b5d82991cfbde5fe5ab3ca50f9c9ceda/Models/CubeVisibility ) was caused by the facts that...

• There was no metadata.json
• There was no README.body.md
• The glTF file names have been cube_visibility.* instead of CubeVisibility.*