Texture properties

[pjcozzi]

Currently, a glTF texture looks like

        "texture_image_0": {
            "format": 6408,
            "internalFormat": 6408,
            "sampler": "sampler_0",
            "source": "image_0",
            "target": 3553
        }

• Do we need to include the pixel datatype, which is the second-to-last argument to texImage2D (GLenum type)?
• When will format !== internalFormat. Do we need to include both?

CC #40

[fabrobinet]

Good points !

• pixel datatype , yes we should include it. But also we could default to UNSIGNED_BYTE (and thus make it optional). What do you prefer ? The reason is just to save some bytes... (provided UNSIGNED_BYTE is common enough to be a default that makes sense.
• I am OK to not include internalFormat if it is equal to format, it just needs to be clear in the spec.

Depending on the decision here, I may have to fix the converter (and add type) or you can mark as resolved.

[pjcozzi]

pixel datatype...we could default to UNSIGNED_BYTE (and thus make it optional)

Agreed. This is the default in our engine and we haven't run into any problems.

I am OK to not include internalFormat if it is equal to format, it just needs to be clear in the spec.

OK.

[fabrobinet]

What's your expectations for the converter flag ? to have this tied to the exportDefault values option ? (which is on by default now).

[pjcozzi]

Yes.

[fabrobinet]

Works for me too. for dev-4

[fabrobinet]

At the moment the converter just output RGBA for format and internalFormat, and UNSIGNED_BYTE for type. So it is simpler than it should be, we'll need to thing about more advanced options (in the JSON config of the converter) to provide more control.

[pjcozzi]

The plan is to drop internalFormat from the spec, right? Can we update the converter for dev-6?

[fabrobinet]

no, it was not the plan. Plan according to comments was to not have it mandatory included when it has same value as format.

[pjcozzi]

Ah, right. OK.