search

imagej / imagej-mesh-io

I/O plugins for ImageJ meshes and related data structures
Other
2 stars 1 forks source link

Read and open PLY files from an InputStream. #3

Closed tinevez closed 1 year ago

tinevez commented 1 year ago

Add methods to read a PLY file from an InputStream.

This will allow playing with meshes stored in e.g. a ZIP file.

For this we needed to rewrite the load logic so that the stream or file is parsed only once. In the original read method that was using a File, the File object was opened and closed several times.

kephale commented 1 year ago

The changes look good, and tests are also running on my machine. I haven't done integration tests with other packages, particularly imagej-mesh. However, I'm not terribly concerned about doing those for these changes. I approve.