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

Merge with geom-parse-obj #2

Open
vorg opened this issue Oct 4, 2016 · 2 comments
Open

Merge with geom-parse-obj #2

vorg opened this issue Oct 4, 2016 · 2 comments

Comments

@vorg
Copy link

vorg commented Oct 4, 2016

Hi, I recently published geom-parse-obj as we needed it for a project. I was a bit reluctant as this package (parse-obj) already exists although there are differences:

  • geom-parse-obj takes a string as an argument
  • it has support for multiple meshes in one file via g sections
  • it focuses only on vertex data and doesn't parse e.g. face normals, just indices
  • it conforms to stack.gl/pex2 naming conventions and returns { positions, normals, uvs, cells }

Given stack.gl, regl and pex ecosystems i think it would be beneficial to have one module that combines the functionality. Is there interest in combining the two?

@mikolalysenko
Copy link
Owner

Sounds good to me. If you want to take over the project I can add you to this module.

@vorg
Copy link
Author

vorg commented Oct 12, 2016

Great. Please add me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants