Conversation
|
Thanks for this @juntyr ! I'm currently trying to test this out. I initially got an error that Also happy to wait to merge #15 until this one is ready! I will get back to #15 tomorrow and try to adjust it based on what we discussed in the meeting. |
Whoops, that's my fault. I'll release a patch release
What are the dimensions of the tiny CAMS dataset? I just tested the codec in the web browser, and it can encode 100x100x100 in <1sec.
We'll merge whichever one is ready first, let's see if we can get this one working |
The tiny CAMS data is (time=4, vertical=4, lat=241, lon=480) so it is roughly the same order of magnitude. At the very least the size difference wouldn't explain the difference in runtime. |
|
So ... the problem was a deadlock. I'm still trying to wrap my head around why, but I know where. I'll try to fix the deadlock next week, for now we can just compress chunks sequentially |
|
@treigerm Could you check that the issue is fixed (after a uv sync) with the new numcodecs-wasm version? Feel free to squash and merge if it is |
|
Can confirm the issue is fixed, thansk @juntyr ! |
Closes #16