vispy / vispy

Main repository for Vispy
http://vispy.org
Other
3.29k stars 617 forks source link

3D mesh in VisPy #1665

Open asnt opened 5 years ago

asnt commented 5 years ago

(Suggested) Ticket to keep track of improvements for 3D mesh rendering (MeshVisual).

Data representation

Rendering

Camera

Lights

Loading

djhoese commented 5 years ago

Maybe vispy 0.7 is the release of mesh updates. The issues from @infosisio are making me realize how much functionality is missing from the MeshVisual and related utilities for displaying .obj (and related) files. I don't personally use Meshes directly in my own work.

@asnt Would you mind joining us on gitter so we can discuss a plan of attack for getting all of these things merged or resolved. I know you have a lot of PRs that are overdue and I'd like get them in, but might need some guidance.

asnt commented 5 years ago

@djhoese v0.7 as target would be great. I'll give a hand where I can.

sofroniewn commented 4 years ago

@djhoese I was wondering what the status of some of these PRs is - namely #1463 - which provides better shading for mesh visuals (and which depends on #1462). I havn't tried any of them out yet, but I think napari would benefit from some nicer shading options for 3D meshes, and #1463 seems like it will provide that.

I am happy to help review / test etc. as needed - just let me know if I can be of any use.

djhoese commented 4 years ago

@sofroniewn A lot of the PRs made by @asnt are very close to being ready to be merged. I have a couple comments in them that I think I'm still waiting on a response for from @asnt.

I appreciate the willingness to help out. If you want to review the various PRs from @asnt, figure out what makes sense, and point out parts that you have concerns about that would be amazing. It is a little difficult to keep track of the various parts of these PRs since I don't really use Meshes directly myself.

sofroniewn commented 4 years ago

Sounds good @djhoese - I'll look at #1462 now as it seems like that is a foundational one

asnt commented 4 years ago

Great to see interest and thanks for your reviews, @sofroniewn @djhoese. I haven't forgotten about these PR's and am still willing to bring them to an acceptable state. I'll try my best in the coming days ;-)

djhoese commented 3 years ago

I checked a couple of the boxes near the top of the main list that PRs were merged for. Feel free to review this @asnt and let me know if I missed any that you've resolved.