Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tooling breaks when "glTF" folder is not present. #129

Open
emackey opened this issue May 29, 2024 · 1 comment
Open

Tooling breaks when "glTF" folder is not present. #129

emackey opened this issue May 29, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@emackey
Copy link
Member

emackey commented May 29, 2024

The tooling here quietly breaks when the glTF folder is not present. Specifically, the tooling issues a warning message to the console during the run, but continues processing anyway. GitHub still places a green checkmark, and it's not obvious anything has gone wrong.

The Playset model added in #116 only includes glTF-Binary and does not include a glTF folder. But the generated README.md for that model fails to list that there are extensions involved:

  "extensionsUsed": [
    "KHR_lights_punctual",
    "KHR_materials_emissive_strength"
  ],

At some point I'm sure we talked about the glTF folder always being desirable, so that we could have at least one variant that was guaranteed to be present for each sample asset, and so that visitors to this repo would not need special tooling of their own to dismantle our sample assets for better understanding. If this is really the case, we should update the CI here to enforce this. If not, we should update the CI to know how to dismantle a GLB and extract the extensions list and any other needed info from it.

@emackey emackey added the bug Something isn't working label May 29, 2024
@DRx3D
Copy link
Contributor

DRx3D commented May 29, 2024

I agree that the current operation is not correct. There was an assumption that all models contain a glTF file in the 'gltf' folder. This is used to obtain the extensions required and used for the model. The extension information has been the subject of repeated requests over the years, so it is reasonable to assume that it is important. I believe that the most reasonable course of action is to find and read the GLB to get that information. In the interim I propose that if the glTF file cannot be found or read, then the extension information in the README indicate that there was an error retrieving that information and continue with the Warning message to the log file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants