Adding Different colors to different meshes in viso3d?

Question

Adding Different colors to different meshes in viso3d?

VimalMollyn opened this issue 2 years ago · comments

Hey! Do you have plans to add support for different colors for each mesh while rendering multiple meshes? I have a hacky version ready, can make a pull request if you want!

Thanks!
Vimal

Vimal Mollyn commented 2 years ago

Yes!

Yuxiao Zhou · Answer 1 · Wed Mar 16 2022 13:21:33 GMT+0800 (China Standard Time)

Hi Vimal,

Thanks for the proposal.
It is great, but I think this may go too far from the goal of this package - wrapping existing methods using minimal code for basic functionalities.
To some extent, this package is trading flexibility for ease of use.
My concern is that if we start to support multiple meshes, with different colors, maybe we should also support changing lights, moving cameras, etc, and things get more and more complicated.
For the highly customized demands, I think it would be better to go to the official open3d, and only left this package for quick tests of ideas.
What do you think?

Best regards,
Yuxiao

Vimal Mollyn · Answer 2 · Thu Mar 17 2022 08:58:10 GMT+0800 (China Standard Time)

Hmm yeah you may be right. I just thought that since the existing package includes support for rendering multiple meshes, adding colors may be helpful in distinguishing them. Maybe we can just include a text label or something else instead?

Yuxiao Zhou · Answer 3 · Thu Mar 17 2022 09:13:16 GMT+0800 (China Standard Time)

Oh, are you referring to visgl? https://github.com/CalciferZh/vctoolkit/blob/master/vctoolkit/visgl/__init__.py#L11

Vimal Mollyn · Answer 4 · Thu Mar 17 2022 10:21:52 GMT+0800 (China Standard Time)

I was referring to viso3d actually.

Yuxiao Zhou · Answer 5 · Thu Mar 17 2022 11:05:39 GMT+0800 (China Standard Time)

Were you referring to render_sequence_3d? https://github.com/CalciferZh/vctoolkit/blob/master/vctoolkit/viso3d/__init__.py#L7

Yuxiao Zhou · Answer 6 · Thu Mar 17 2022 21:07:09 GMT+0800 (China Standard Time)

This function was meant to render a sequence with a changing mesh and the first axis of verts is for the number of frames instead of multiple meshes. Guess you found a different way of using it :)