-
Notifications
You must be signed in to change notification settings - Fork 33
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
meshio for mesh I/O #51
Comments
Ah good point! I didn't bother hunting for existing implementations for neuroglancer's "precomputed" format. I have independent functions in various other packages and this was actually my attempt at consolidating those into navis. At a glance it looks like
I guess the latter two could be contributed back upstream to |
Yes, it was a while ago that I added that so not surprised if the format as come on since then. As it is such a simple format, it also doesn't surprise that the dependency (which is pretty light without extras) wouldn't save much code, but it would add quite a lot of flexibility to navis for reading and writing different mesh formats. |
Oh, I see. We might be thinking of slightly different things here. I'm not at all opposed to adding it as a dependency but perhaps you can elaborate a bit on how you would suggest proceeding? |
Spotted some commits implementing more mesh formats - would it be worth just (optionally?) depending on meshio and calling it a day? Improvements and additional formats could be contributed back upstream. It has neuroglancer's mesh format from a while back - reducing the number of implementations out in the wild is probably for the best.
The text was updated successfully, but these errors were encountered: