Should image have a name property?

[pjcozzi]

Same question as #234 but for an image.

"images": {
        "image_0": {
            "path": "duckCM.png"
        }
    }

When I first did the schema I probably carried this over from COLLADA. Do we want to keep a name property here?

[pjcozzi]

Same question for shaders:

"shaders": {
        "duck0FS": {
            "path": "duck0FS.glsl",
            "type": 35632
        }

[pjcozzi]

@fabrobinet thoughts?

[fabrobinet]

I would that allowing our top level entities meshes , shaders , programs ... to all have a name that is optional doesn't bring much risk (I mean for legacy) and may be very appreciated when showing informations to users in Viewers UI...

[fabrobinet]

... so it is a yes for image. (but honnestly, I have not a strong opinion on that one).

[pjcozzi]

Agreed. I'll update the schema now.

[tparisi]

My $.02: there is no harm in keeping name fields for these assets... and there might be harm in taking them out. So I say yes.

[pjcozzi]

Schema is updated. ca69d69

@fabrobinet do we want to keep this open and labled as converter to make any changes to carry through the COLLADA names?

[fabrobinet]

I'll keep this track that all names are propagated in the converter. (and not only image)

[tfili]

Should we be doing this with lights and cameras as well?

[tparisi]

yes -

On Thu, Oct 1, 2015 at 4:46 PM, Tom Fili notifications@github.com wrote:

Should we be doing this with lights and cameras as well?

—
Reply to this email directly or view it on GitHub
#235 (comment).

Tony Parisi tparisi@gmail.com
Follow me on Twitter! http://twitter.com/auradeluxe
Read my blog at http://www.tonyparisi.com/
Learn WebGL http://learningwebgl.com/
Mobile 415.902.8002
Skype auradeluxe

Read my books!
Learning Virtual Reality
http://www.amazon.com/Learning-Virtual-Reality-Experiences-Applications/dp/1491922834

Programming 3D Applications in HTML5 and
WebGLhttp://www.amazon.com/Programming-Applications-HTML5-WebGL-Visualization/dp/1449362966
http://www.amazon.com/Programming-Applications-HTML5-WebGL-Visualization/dp/1449362966WebGL,
Up and Running
http://www.amazon.com/dp/144932357X

[tfili]

Added name property to lights, cameras and images in dev-1.0 branch at 1d4370f

[pjcozzi]

@tfili did you also confirm that all names are propagated from COLLADA to glTF?

What about ids (#9)?

[pjcozzi]

@tfili did you see this:

did you also confirm that all names are propagated from COLLADA to glTF?

[tfili]

@pjcozzi I'll double check when I get home.

[tfili]

@pjcozzi Looks like skins didn't have it but I added it with 2e9edf0

[pjcozzi]

Great, thanks @tfili!

[tparisi]

thanks @tfili