Wouter-Michiel Vierdag

Results 227 comments of Wouter-Michiel Vierdag

> I like the new wording better! > > However, this part is still unclear to me: > > ``` > print(v.export_figure(scale=1).shape) > print(v.export_figure(scale=None).shape) > ``` > > These _shoudl...

> Discussed at the PR party, we'd like @melonora to weigh in also. I did have another look. The approach that @Czaki has suggested has been implemented in the meantime...

I did, the zarr store with current version of `spatialdata` and `anndata` version 0.10.3 creates an `_index` zarr array in the `var` zgroup. This is different from what we observe...

@LucaMarconato is the uploaded steinbock date perhaps outdated?

This is the anndata io by spatialdata. With the merfish example when I adjust the var names I get a var zarr group with this: ![afbeelding](https://github.com/scverse/anndata/assets/22346363/24325f65-4c8d-43f1-bf6d-2cefdd1b942f) Reading it back in...

Yeah the behaviour is really different. The encoding_type of var in this case is a dataframe. This is specified in .zattrs in var.

This gives an error because of the `/` being interpreted as a path separator. The thing that I find weird is that the issue arises with the Steinbock dataset where...

To be specific this raises a `FileNotFoundError` as a result of a path with forward slashes not being found: ![afbeelding](https://github.com/scverse/anndata/assets/22346363/d6625a92-8bdf-4abe-8c18-42a2ffbe8792)

Completed with [#596](https://github.com/scverse/spatialdata/pull/596)

There is PR #971 that fixes the remote storage completely; however, between that PR and zarrv3 being merged, zarrv3 got merged first. So #971 would require an update.