Repository navigation
Handle bounding boxes in 2D ndimage warping - #117
sylvesterkaczmarek wants to merge 1 commit into
Conversation
|
@mjanusz This PR is current with main and has no public failing checks; only the Google import/review gate remains. It adds the 2D ndimage bounding-box handling with focused coverage. Could you review the current head when convenient? |
720a2c0 to
bfa6f38
Compare
|
@mjanusz Refreshed onto current |
|
Could a maintainer review the 2D bounding-box warping fix and unblock the workflow approval and Copybara import? |
ndimage_warpassumes three spatial dimensions when reshaping map offsets, allocating an explicit output box and computing map/output offsets. With 2D images, explicit bounding boxes cause broadcasting errors or add an unwanted axis.Make those operations dimension-aware. Add 2D regressions and 3D controls with translated/cropped boxes, unequal strides, output scaling and threaded warping.
Validation:
python -m pytest -q tests/warp_test.py tests/map_utils_test.py tests/flow_utils_test.pypasses all 32 tests. Five new 2D regression cases fail on unchanged upstream; all five 3D controls already pass. Syntax parsing andgit diff --checkpass.