_getitem_* functions must consistently set the slicer in handle_error. #1469

jpivarski · 2022-05-13T20:38:01Z

No description provided.

codecov · 2022-05-13T21:05:43Z

Codecov Report

Merging #1469 (672fe12) into main (de7cae8) will decrease coverage by 0.44%.
The diff coverage is 70.19%.

Impacted Files	Coverage Δ
src/awkward/_v2/_connect/numexpr.py	`88.40% <ø> (ø)`
...c/awkward/_v2/_connect/rdataframe/to_rdataframe.py	`0.00% <0.00%> (ø)`
src/awkward/_v2/operations/ak_backend.py	`10.00% <0.00%> (ø)`
src/awkward/_v2/operations/ak_from_arrow.py	`100.00% <ø> (ø)`
src/awkward/_v2/operations/ak_from_arrow_schema.py	`50.00% <ø> (ø)`
src/awkward/_v2/operations/ak_from_buffers.py	`86.13% <ø> (ø)`
src/awkward/_v2/operations/ak_from_cupy.py	`50.00% <ø> (ø)`
src/awkward/_v2/operations/ak_from_jax.py	`50.00% <ø> (ø)`
src/awkward/_v2/operations/ak_from_json_new.py	`0.00% <0.00%> (ø)`
src/awkward/_v2/operations/ak_from_numpy.py	`100.00% <ø> (ø)`
... and 110 more

_getitem_* functions must consistently set the slicer in handle_error.

672fe12

jpivarski marked this pull request as ready for review May 13, 2022 20:43

jpivarski enabled auto-merge (squash) May 13, 2022 20:43

jpivarski merged commit 9402988 into main May 13, 2022

jpivarski deleted the jpivarski/some-getitem-clean-ups branch May 13, 2022 21:08