Jeremy Maitin-Shepard

> I get errors saying: > > ``` > docs\apidoc\cpp\index.rst:23: WARNING: cpp:identifier reference target not found: ::my_ns3 > docs\apidoc\cpp\index.rst:25: WARNING: cpp:identifier reference target not found: ::my_ns3 > ``` > >...

Thanks, this is a great improvement, and we can iterate on it further in the future if needed.

Both issues are now fixed --- tensorstore now compiles with VS 2022 17.5, and also all of the tests compile in Release mode.

zarr vs n5 performance will be very similar. In general I would recommend zarr over n5 since zarr is becoming more standard and offers a more functionality. Compared to zarr-python,...

Closing for now since this is 4 years old and was unable to reproduce it. Please reopen if it is still an issue.

You could certainly take a stab at implementing it. The closest existing driver is the neuroglancer_uiny64_sharded driver: https://github.com/google/tensorstore/blob/master/tensorstore/kvstore/neuroglancer_uint64_sharded/neuroglancer_uint64_sharded.cc And we'd be happy to provide guidance. Due to the need to...

TensorStore already depends on zlib, bzip, and liblzma, so supporting those compression methods would not require any additional dependencies. To integrate minizip-ng we would need a bazel build for it,...

What I had in mind was to just implement the zip support "from scratch" (but still using zlib, etc. for the actual compression). Basically for reading: - Locate and decode...

Yes, you could make a simpler version that always reads the entire zip file into memory. Then you could easily use an existing library like minizip-ng. To add support to...

This commit (not pushed to the main branch) shows how it can be done: https://github.com/jbms/tensorstore/commit/5d8ce407e3be5ade26f1976b3d2d39d551463874 This seems to work on Linux, I haven't tested on Windows but you can probably...