DracoPy icon indicating copy to clipboard operation
DracoPy copied to clipboard

v0.0.18 | Supports Texture Coordinates and Normals

Open farunurisonmez opened this issue 3 years ago • 6 comments

#5 Can it encode normal or uv?

  • [x] Decode supports Texture Coordinates and Normals
  • [ ] Encode supports Texture Coordinates and Normals

Co-Authored-By: Faru Nuri Sönmez [email protected]

farunurisonmez avatar May 07 '21 20:05 farunurisonmez

Hi Faru!

I'm the new maintainer for this repo and I'll be happy to review your PR! I'm a bit less experienced with the Draco spec compared with the library creator, so I might need some tips to help review. ^_^

I'll leave some comments on the PR. Would you mind also adding an automated test?

Thanks!! Sorry it took so long to get this reviewed.

william-silversmith avatar Oct 22 '21 22:10 william-silversmith

Thanks again for this valuable contribution!

william-silversmith avatar Oct 22 '21 22:10 william-silversmith

Hi Faru! I adopted some of your changes (the normal decode) and gave you a credit in the setup.py file! I was working rapidly so I didn't adopt the textures as I wasn't sure how to test it. Thanks for your contribution!

william-silversmith avatar Dec 14 '21 07:12 william-silversmith

Sorry, closed by mistake when I meant to comment.

william-silversmith avatar Dec 14 '21 07:12 william-silversmith

@william-silversmith Thank you very much for further upgrading the library, would it be possible to add UV / texture support as well? If you need an example script / files I could provide it to you for testing.

cansik avatar Apr 09 '22 16:04 cansik

Hi Florian, I'd be happy to consider adding it. A script and data would help tremendously!

On Sat, Apr 9, 2022, 12:05 PM Florian Bruggisser @.***> wrote:

@william-silversmith https://github.com/william-silversmith Thank you very much for further upgrading the library, would it be possible to add UV / texture support as well? If you need an example script / files I could provide it to you for testing.

— Reply to this email directly, view it on GitHub https://github.com/seung-lab/DracoPy/pull/17#issuecomment-1094075456, or unsubscribe https://github.com/notifications/unsubscribe-auth/AATGQSJKC3USLB6B7XANIADVEGTFFANCNFSM44K7IY7Q . You are receiving this because you were mentioned.Message ID: @.***>

william-silversmith avatar Apr 09 '22 18:04 william-silversmith