You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently earthaccess.open_virtual_mfdataset is implementing some logic that should be in virtualizarr. We should remove this logic from earthaccess and simplify the current implementation. Then the earthaccess API will only add logic for unwrapping results and logic for a few convenience params like load and access. The rest of the arguments will be directly passed to virtualizarr. The change to open_virtual_mfdataset needs virtualizarr.open_virtual_mfdataset to be implemented upstream. cc @betolink@TomNicholas
Currently
earthaccess.open_virtual_mfdataset
is implementing some logic that should be in virtualizarr. We should remove this logic from earthaccess and simplify the current implementation. Then the earthaccess API will only add logic for unwrapping results and logic for a few convenience params likeload
andaccess
. The rest of the arguments will be directly passed to virtualizarr. The change toopen_virtual_mfdataset
needsvirtualizarr.open_virtual_mfdataset
to be implemented upstream. cc @betolink @TomNicholasHere is my suggested API change:
The text was updated successfully, but these errors were encountered: