Closed johnkerl closed 1 month ago
@mojaveazure ready for round two!
Attention: Patch coverage is 78.57143%
with 6 lines
in your changes missing coverage. Please review.
Project coverage is 89.79%. Comparing base (
ff2eb5b
) to head (68b9767
). Report is 7 commits behind head on main.
@nguyenv and @mojaveazure ready for the next round! :)
Issue and/or context: As tracked on issue #2407 / [sc-51048]. Here we implement
resize
andupgrade_shape
for ND arrays. The dataframe mutators are deferred to a separate PR.Note that the intended Python and R API changes are all agreed on and finalized as described in #2407.
Changes:
Implements and tests the mutators.
Notes for Reviewer:
For a separate PR: needs unit-test cases, including some saved-off arrays (in a tar file, say) lacking the new-shape feature which we can apply upgrade-shape to in unit-test cases.