HelenaLC / SpatialData

14 stars 7 forks source link

trimming and passing check #55

Open vjcitn opened 1 week ago

vjcitn commented 1 week ago

I have forked the package and will remove the merfish.zarr from extdata. I will issue a PR once I have introduced a function that retrieves the necessary data from Bioc Open Storage Network and caches it in BiocFileCache.

I will not PR until the package is passing check, but I hope that can happen soon.

vjcitn commented 1 week ago

I would propose that we avoid including all but the smallest demonstration data elements to our packages. The alabaster/gypsum APIs should be looked at for achieving language-agnostic data repositories, but we are not at a point where it is clear how to adopt them.

HelenaLC commented 1 week ago

Agreed. The idea is to instead include examples from here, however, I am awaiting these to be updated. Alternatively, once we have something functional, we could mock up & write out minimal example data, because it still would be nice to include a www-independent example, I think.

vjcitn commented 1 week ago

I have a bucket already set up. We can use it until we decide on a canonical longer-term solution.