Giter VIP home page Giter VIP logo

Comments (7)

fabrobinet avatar fabrobinet commented on August 26, 2024

Patrick initial question was: "Do all COLLADA models have indices or do we need to adjust the statement about drawArrays above? Or make changes to the converter?" . I would say YES.

from gltf.

fabrobinet avatar fabrobinet commented on August 26, 2024

I mean YES COLLADA models have indices, but we still need to decide if we adjust.

from gltf.

pjcozzi avatar pjcozzi commented on August 26, 2024

Although I also have use cases with points, triangle strips, triangle fans, line strips, etc., I am OK with requiring indices until we get significant community feedback otherwise. Here the current language I have in the spec:

To reduce burden on the application developer, glTF does not include all the flexibility of the GL APIs. Features not relavant to assets or in widespread use are not included, especially if they can be supported in the content pipeline. For example, in glTF:

  • ...
  • Geometry is defined with indices, implying drawElements, not drawArrays, should be used to issue draw calls. This is because most assets are defined with indexed triangle lists and triangle soup, strips, and fans can be converted to indexed triangle lists in the content pipeline.

from gltf.

pjcozzi avatar pjcozzi commented on August 26, 2024

Since the spec already states that glTF requires indices, can we close this? #60 tracks updates to the converter.

from gltf.

fabrobinet avatar fabrobinet commented on August 26, 2024

Indeed, this issue as been around and discussed for a while, we agreed since the beginning on requiring indices. Let's close.

from gltf.

mlimper avatar mlimper commented on August 26, 2024

requiring indices until we get significant community feedback otherwise. Here the current language I have in the spec:

Here's some community feedback :-)

Since release 1.0 is coming closer, I just wanted to ask if it really makes sense to force the "indices" property?

It would help to use non-indexed data even for some cases where you render triangles, for example when using flat shading (since the per-face normals require you to duplicate vertices anyway, you'll end up without any re-use of data).
This seems like a quick win, by just defining indices to be an optional property... or would that cause problems?

from gltf.

pjcozzi avatar pjcozzi commented on August 26, 2024

I agree that indices should not be required. I think we need to tweak the spec a bit, but I haven't looked closely. See #60 (comment)

Let's continue the conversation in that issue.

from gltf.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.