<div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">niedz., 9 sie 2020 o 01:10 Michalis Kamburelis <<a href="mailto:michalis.kambi@gmail.com">michalis.kambi@gmail.com</a>> napisał(a):<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>I looked at the new prose that requires support for glTF in Inline in X3D. Comments:<br></div><div><br></div><div>1. What is the goal of level 4 of the "Lighting" component, <a href="https://www.web3d.org/specifications/X3Dv4Draft/ISO-IEC19775-1v4-WD2/Part01/components/lighting.html" target="_blank">https://www.web3d.org/specifications/X3Dv4Draft/ISO-IEC19775-1v4-WD2/Part01/components/lighting.html</a> ? As far as I can see, level 3 of this component already includes everything -- all nodes (3 light nodes with all fields supported). I don't know what is different between level 3 and 4.<br></div><div><br></div><div>2. Level 4 of the "Networking" component (bottom of <a href="https://www.web3d.org/specifications/X3Dv4Draft/ISO-IEC19775-1v4-WD2/Part01/components/networking.html" target="_blank">https://www.web3d.org/specifications/X3Dv4Draft/ISO-IEC19775-1v4-WD2/Part01/components/networking.html</a> ) now says this:</div><div><br></div><div>    """<br></div><div><span title="proposed changes to Inline content, security precautions">    Support for .gltf (<code>model/gltf+json</code>) and .bin (<code>application/octet-stream</code>).</span>
            <br>   
            <span title="proposed changes to Inline content, security precautions">Requires support for Shape component level 2 and Lighting component level 4.</span></div><div><span title="proposed changes to Inline content, security precautions">    """<br></span></div><div><span title="proposed changes to Inline content, security precautions"><br></span></div><div><span title="proposed changes to Inline content, security precautions">    My notes:</span></div><div><span title="proposed changes to Inline content, security precautions"></span></div><br><div><span title="proposed changes to Inline content, security precautions">    - The .bin extension (and MIME type) is not standardized, and it is also outside of scope for this X3D prose, IMHO. "Inline" node in X3D will *not* refer to .bin files. The glTF contents refer to .bin files. IOW, the support for external binary files (with whatever extension) is implicit in "supporting glTF".<br></span></div><div><span title="proposed changes to Inline content, security precautions"><br></span></div><div><span title="proposed changes to Inline content, security precautions">    - If you wanted to include the "packed" glTF files, then you probably wanted to instead mention .glb ("model/gltf-binary" MIME type). Possibly you mistakenly confused the .glb and .bin files.<br></span></div><div><span title="proposed changes to Inline content, security precautions"><br></span></div><div><span title="proposed changes to Inline content, security precautions">    - This clause is too imprecise, IMHO. What exactly glTF features do we require to be supported? As you saw in my document on <a href="https://github.com/michaliskambi/x3d-tests/wiki/Converting-glTF-to-X3D" target="_blank">https://github.com/michaliskambi/x3d-tests/wiki/Converting-glTF-to-X3D</a> , we have straightforward way to convert many things from glTF to X3D nodes, but not 100% of glTF. There are lots of details that still need to be filled to express 100% of glTF in X3D. Even some things I mention on my wiki are right now just extensions of CGE or X3DOM, which are not (yet?) in X3D spec. This is a work in progress, and I suppose it will happen across multiple implementations.  <br></span></div><div><span title="proposed changes to Inline content, security precautions"><br></span></div><div><span title="proposed changes to Inline content, security precautions">My proposal:</span></div><div><span title="proposed changes to Inline content, security precautions"><br></span></div><div><span title="proposed changes to Inline content, security precautions">AD 1 - Remove level 4 of the "Lighting" component. It doesn't seem needed.</span></div><div><span title="proposed changes to Inline content, security precautions"><br></span></div><div><span title="proposed changes to Inline content, security precautions">AD 2 - Change the level 4 of the "Networking" component text, to require this:<br></span></div><div><span title="proposed changes to Inline content, security precautions"><br></span></div><div><div>    """<br></div><div><span title="proposed changes to Inline content, security precautions">    Support for glTF models in Inline nodes, with .gltf (model/gltf+json) and .glb (model/gltf-binary) formats.</span></div><div><span title="proposed changes to Inline content, security precautions">    Minimum required glTF support:</span></div><div><span title="proposed changes to Inline content, security precautions">    - glTF transformation hierarchy (glTF "nodes", which correspond to X3D "Transform" nodes),</span></div><div><span title="proposed changes to Inline content, security precautions">    - glTF meshes,</span></div><div><div><span title="proposed changes to Inline content, security precautions">    - glTF standard physical materials (correspond to X3D PhysicalMaterial nodes, requires "Shape" component level 2 support)</span></div><div><span title="proposed changes to Inline content, security precautions">    - glTF unlit (KHR_materials_unlit) materials (correspond to X3D UnlitMaterial nodes)<br></span></div><div><span title="proposed changes to Inline content, security precautions">    - loading of external binary data referenced from .gltf files (e.g. for vertex coordinates)<br></span></div><div><span title="proposed changes to Inline content, security precautions">    """<br></span></div><div><span title="proposed changes to Inline content, security precautions"><br></span></div><span title="proposed changes to Inline content, security precautions"></span></div></div><div>Regards,</div><div>Michalis<br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">sob., 1 sie 2020 o 20:22 Don Brutzman <<a href="mailto:brutzman@nps.edu" target="_blank">brutzman@nps.edu</a>> napisał(a):<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Discussed briefly with Dick and Michalis.  Also added requirement for corresponding level 4 in Lighting component.<br>
<br>
On 8/1/2020 8:05 AM, Don Brutzman wrote:<br>
> Important gap: we don't specifically call out glTF under Inline node in Networking component.<br>
> <br>
> Existing prose:<br>
> ===============<br>
> 9.4.2 Inline<br>
> [...]<br>
> The run-time system can support any number of 3D model resource types as long as those follow the abstract model definition (see 2.[RFC2077]), provide a registered content type (e.g. model/x3d-xml, model/gltf-bin, model/stl, etc.), and can be determined with some form of content negotiation (see 2.[RFC2616]). The run-time system must support at least one X3D type (e.g. model/x3d-xml) but can also support and negotiate any number of X3D encodings and (optionally) non-X3D representation formats.<br>
> ===============<br>
> <br>
> Recommended additions:<br>
> <br>
> "Support for loading glTF assets also requires support for Shape component level 3."<br>
> <br>
> Support levels:<br>
> <br>
>      4<br>
>      Model support<br>
>      Support for .gltf (model/gltf+json) and .bin (application/octet-stream)<br>
> <br>
> Normative references<br>
> <br>
> * GLTF<br>
> * GL Transmission Format (glTF) Specification, The Khronos Group, Version 2.0, 2017.<br>
>    <a href="https://github.com/KhronosGroup/glTF/tree/master/specification/2.0" rel="noreferrer" target="_blank">https://github.com/KhronosGroup/glTF/tree/master/specification/2.0</a><br>
> <br>
> Related reference section for glTF MIME information:<br>
> <br>
> [1] glTF File Extensions and MIME Types<br>
>      <a href="https://github.com/KhronosGroup/glTF/tree/master/specification/2.0#file-extensions-and-mime-types" rel="noreferrer" target="_blank">https://github.com/KhronosGroup/glTF/tree/master/specification/2.0#file-extensions-and-mime-types</a><br>
> <br>
> All review and potential improvement of this important section is welcome.<br>
> <br>
> all the best, Don<br>
<br>
all the best, Don<br>
-- <br>
Don Brutzman  Naval Postgraduate School, Code USW/Br       <a href="mailto:brutzman@nps.edu" target="_blank">brutzman@nps.edu</a><br>
Watkins 270,  MOVES Institute, Monterey CA 93943-5000 USA   +1.831.656.2149<br>
X3D graphics, virtual worlds, navy robotics <a href="http://faculty.nps.edu/brutzman" rel="noreferrer" target="_blank">http://faculty.nps.edu/brutzman</a><br>
</blockquote></div>
</blockquote></div>