refactor!: retrieve and decode _into APIs

[LDeakin]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 91.51625% with 47 lines in your changes missing coverage. Please review.

Project coverage is 81.90%. Comparing base (f675f06) to head (9bd09fb).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
zarrs/src/array/codec.rs	46.42%	15 Missing ⚠️
...s/src/array/codec/array_partial_encoder_default.rs	0.00%	7 Missing ⚠️
...ay/codec/array_to_array_partial_encoder_default.rs	0.00%	7 Missing ⚠️
zarrs/src/array/array_bytes.rs	91.89%	6 Missing ⚠️
zarrs/src/array/array_bytes_fixed_disjoint_view.rs	97.75%	5 Missing ⚠️
...ay/codec/array_to_bytes/sharding/sharding_codec.rs	91.37%	5 Missing ⚠️
zarrs/src/array/array_sync_readable.rs	96.00%	1 Missing ⚠️
...arrs/src/array/codec/array_to_bytes/codec_chain.rs	93.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #136      +/-   ##
==========================================
- Coverage   81.93%   81.90%   -0.03%     
==========================================
  Files         171      172       +1     
  Lines       24268    24340      +72     
==========================================
+ Hits        19883    19935      +52     
- Misses       4385     4405      +20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[flying-sheep]

Really cool, that makes all of zarrs-python’s use almost safe.

there we take items from python and convert them into (non-overlapping) chunk_item::WithSubsets, so we can just add checks to that conversion so the checking is next to the unsafe code.

or we could move this use case to zarrs, seems like a thing people would like to do.