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

Add fastgltf #155

Merged
merged 1 commit into from
Sep 26, 2022
Merged

Add fastgltf #155

merged 1 commit into from
Sep 26, 2022

Conversation

spnda
Copy link
Contributor

@spnda spnda commented Sep 25, 2022

I also noticed that the "import" and "load" tasks seem to have the same meaning, is that known?

@CLAassistant
Copy link

CLAassistant commented Sep 25, 2022

CLA assistant check
All committers have signed the CLA.

@javagl
Copy link
Contributor

javagl commented Sep 26, 2022

I also noticed that the "import" and "load" tasks seem to have the same meaning, is that known?

There is an issue talking about the exact meaning of "task" and its possible values: #111 . The issue also explains a bit of where the current categories came from.

One difficulty is that it's hard to assign a "strong" meaning to these keywords: They don't have semantics that all developers can agree on. A library developer might call a function readGltf(uri), or loadGltf(uri), or importGltf(uri). The potential differences in the meaning are subtle and maybe even subjective. And on a technical level, this even might depend on the context - e.g. whether this is the implementation of an existing Reader, Loader or Importer interface of another library.

For me, "load" sounds a little bit more bare-boned, like in a library that only reads the glTF data structures, whereas "import" sounds more like something that happens in the context of an application - e.g. to import a glTF into Blender or so. But this is highly subjective. Maybe we can clarify that in the context of the other issue.

@javagl javagl merged commit f76f70e into KhronosGroup:main Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants