Closed johnkerl closed 1 month ago
Attention: Patch coverage is 78.43137%
with 11 lines
in your changes missing coverage. Please review.
Project coverage is 61.31%. Comparing base (
8c4890c
) to head (5312d49
). Report is 2 commits behind head on main.
@mojaveazure can you please take a look at the R side? Thank you!
Thanks @nguyenv and @mojaveazure !
Issue and/or context: As tracked on issue #2407 / [sc-51048].
Note that the intended Python and R API changes are all agreed on and finalized as described in #2407.
Changes:
_maybe_sjid_shape
and_maybe_sjid_maxshape
accessors will be crucial for experiment-level resize logic as on append mode, which does not yet exist in TileDB-SOMA-R (see #1630)Notes for Reviewer: