cupy icon indicating copy to clipboard operation
cupy copied to clipboard

ndimage: more efficient testing of binary morphology functions

Open grlee77 opened this issue 1 year ago • 2 comments

The purpose of this MR is to refactor existing binary morphology tests from a single dense product over all parameters to 2 or 3 products over a small subset of the parameters. The functionality tested should be equivalent while reducing the total number of test cases run by an order of magnitude.

When running the morphology tests with pytest flag -k Binary, the number of tests that run is reduced from 4054 (before) to 412 (after).

grlee77 avatar Apr 28 '24 19:04 grlee77

/test mini

takagi avatar May 01 '24 05:05 takagi

This pull request is now in conflicts. Could you fix it @grlee77? 🙏

mergify[bot] avatar May 01 '24 08:05 mergify[bot]

conflicts have been resolved, please retest

grlee77 avatar May 03 '24 13:05 grlee77

/test mini

takagi avatar May 08 '24 09:05 takagi