react-three-renderer Added 'uv' and 'groups' support to BufferGeometryDescriptor

This fork/branch adds the possibility to use BufferGeometry with UVs and Groups (although assigning different materials for each group is another story).

Sep 07 '17 20:09 chiguireitor

Thanks for the PR :) I think we may need to test to see what happens if for example 3 groups are used first and then that's changed to 2 groups, but I can do that in the merge branch, I think I get the main idea of the PR already :D

Sep 18 '17 05:09 toxicFork

Excellent, i'm more concerned about UVs than groups, if you want i can separate that in couple of PRs

Sep 18 '17 14:09 chiguireitor

Yes having a separate PR for them will make it easier to press the merge button :)

Sep 18 '17 19:09 toxicFork

Should keep the PRs against master or do you have a development branch?

Sep 18 '17 19:09 chiguireitor

Master is good

Sep 18 '17 19:09 toxicFork

I'll add in uvs in a commit, for groups that will take some more effort

Nov 10 '17 08:11 toxicFork

Yes thanks, sorry for not separating the PRs, project's taking all my time and i have it 'npm link' on my rig X-D

Nov 10 '17 11:11 chiguireitor

Don't worry, I have similar issues :D we need an open source support group

On Fri, 10 Nov 2017, 11:51 Chiguireitor, [email protected] wrote:

Yes thanks, sorry for not separating the PRs, project's taking all my time and i have it 'npm link' on my rig X-D

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/toxicFork/react-three-renderer/pull/194#issuecomment-343454052, or mute the thread https://github.com/notifications/unsubscribe-auth/AA0iLVSjq6g8h-TyVpPgja72DTblQNB-ks5s1DjYgaJpZM4PQV9F .

Nov 10 '17 11:11 toxicFork