search

keller-mark / pizzarr

Slice into Zarr arrays in R 🍕
https://keller-mark.github.io/pizzarr/
MIT License
25 stars 2 forks source link

Interest in rebooting development on this? #9

Closed dblodgett-usgs closed 10 months ago

dblodgett-usgs commented 11 months ago

User story

As an R developer working with NetCDF data stored in ZARR, I need a low level interface to layer NetCDF metadata interpretation code from https://hypertidy.github.io/ncmeta/ and similar packages such as https://github.com/DOI-USGS/ncdfgeom and https://github.com/r-spatial/stars

Preferred solution

A group of motivated users pushes this package across the finish line and wraps it in an API that can be called with nearly the same function signatures as the https://github.com/mjwoods/RNetCDF package.

Seems like there's a solid start here. @keller-mark -- do you think you have any space to reboot work on this? I could make some contributions and maybe twist some arms from around the community to do the same.

keller-mark commented 11 months ago

Hi @dblodgett-usgs

If you have not seen Rarr you should check out https://github.com/grimbough/Rarr as it seems to be further along than this implementation.

Regardless, I will try to find time to work on this again if for nothing else than to experiment with the possibility of a more R-based implementation. I also tried to reflect the zarr-python API here which could be another differentiator.

dblodgett-usgs commented 10 months ago

Thanks for the pointer -- that's interesting.

I'm new to R6 and really come to this from the landscape / climate data domain.

I see a comment over at Rarr in an issue calling for a C++ ZARR implementation that could be wrapped in an R package. I also see the logic in what they've done with Rarr -- just focusing on a core (non hierarchical) array. That seems sensible but I still think it's important to get ZARR going in base R in a complete way.

Given that, I'd love to find time to support what you are doing here. Are you interested in taking PRs? I know that can just cause more work and headaches given that others are going to have slightly different coding style and such.

Cheers!

keller-mark commented 10 months ago

Hi @dblodgett-usgs

Happy to accept PRs - I just merged a few PRs but I am going to take another break. I made some new issues if you are interested to take any on.

dblodgett-usgs commented 10 months ago

It's been awesome to see this start moving again! I am trying to arrange some time for me to contribute some time to this over the winter. 🙏 for all your work!!!

keller-mark commented 10 months ago

Sounds great! It should be more straightforward to contribute now that a few things are at least functional!