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

GLTF support #544

Closed
FaberSanZ opened this issue Jul 9, 2021 · 7 comments
Closed

GLTF support #544

FaberSanZ opened this issue Jul 9, 2021 · 7 comments
Assignees
Labels
enhancement New feature or request

Comments

@FaberSanZ
Copy link

Hi, any plans for add GLTF?
My loader that uses System.Text.Json and an internal generator that I can donate.

@FaberSanZ FaberSanZ added the enhancement New feature or request label Jul 9, 2021
@HurricanKai
Copy link
Member

At the moment we only provide bindings for native libraries, but I actually think it might not be too bad of an idea to have a GLTF parser. @Perksey thoughts?

@Perksey
Copy link
Member

Perksey commented Jul 9, 2021

Stuff like this is something I’d like in the library, I do want to make Silk.NET more of a toolbox and actually be “the one library you’ll need” for graphics applications.

A fully managed model loading library is something I can see being part of the Silk.NET suite, but we’d need to put work into the design to verify that it makes us all happy if we wanted to bring it under our umbrella.

We’ll be having a discussion on utilities like this in the next working group meeting (among other things), where if you’d like we can discuss this with our stakeholders on what role Silk.NET should play (and if utilities like this fits it) if you’d like.

Working group meetings happen at ~7pm BST and we announce the date in Discord (also where the meeting takes place) ahead of time.

@FaberSanZ
Copy link
Author

Assimp supports importing some data from glTF, I think it is better to keep GLTF as separate library from other formats, Godot

@Perksey
Copy link
Member

Perksey commented Aug 3, 2021

We will be discussing this in the Working Group meeting on Thursday 5th August at 7pm BST in our Discord server. Feel free to come along and discuss with us.

@Perksey Perksey added this to the Next Working Group Meeting milestone Aug 3, 2021
@Perksey Perksey self-assigned this Aug 3, 2021
@HurricanKai HurricanKai self-assigned this Aug 3, 2021
@HurricanKai
Copy link
Member

HurricanKai commented Aug 3, 2021

I'm very happy to advocate for this and eventually implement GLTF if no external contributor will :)

@Perksey
Copy link
Member

Perksey commented Aug 5, 2021

Discussed in Working Group meeting Thursday 5th August 7:45pm-7:55pm BST:

  • Would gain updates as part of our support schedule
  • Not a binding, just another "useful library" (see VMA support #496 on our discussions on this)
  • There are concerns for staying up-to-date
    • doesn't change often
  • We will always have new standards and libraries to add support, again see VMA support #496 for the "where is the line"
    • We just end up exploding the library into lots of maintainable things.
  • There seems to be an official loader
    • potentially not performant (Newtonsoft.Json etc)

Consensus:

  • We won't have one in the main library right now, keep it a community project
  • Seriously consider silk-community org in a future meeting.

@Perksey
Copy link
Member

Perksey commented Aug 10, 2021

Going to close this off now, thanks for the offer for donating your library! In future meetings we'll be discussing an organisation for community projects which have our seal of approval but neither the Silk.NET team or the .NET Foundation supervise or directly run i.e. we'll recommend them to users and they'll see that we approve of the library, but they won't be bound by our support schedule and won't have the strong backing the Foundation provides.

@Perksey Perksey closed this as completed Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants