Changes test_func_order tests for some unary functions

oleksandr-pavlyk commented 2 months ago

The change is to iterate over permutations first, doing some work that does not depend on the order, and then iterate over orders.

This should save doing some computations 4 times (once for each order), now only done once.

The change shaves 4 seconds off of execution of entire test suite on CPU device (from 5 minutes 6 seconds to 5 minutes and 2 seconds).

[x] Have you provided a meaningful PR description?
[ ] Have you added a test, reproducer or referred to an issue with a reproducer?
[x] Have you tested your changes locally for CPU and GPU devices?
[x] Have you made sure that new changes do not introduce compiler warnings?
[x] Have you checked performance impact of proposed changes?
[ ] If this PR is a work in progress, are you opening the PR as a draft?

github-actions[bot] commented 2 months ago

Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. :crossed_fingers:

coveralls commented 2 months ago

coverage: 87.892%. remained the same when pulling cda457456b9d043a58e924f65d8805024ccbb599 on optimize_tests_order into 6ae9f3d6f906dc9d4ff0c63a86d67687d60eea4a on master.

github-actions[bot] commented 2 months ago

Array API standard conformance tests for dpctl=0.17.0dev0=py310h15de555_318 ran successfully. Passed: 870 Failed: 8 Skipped: 92

IntelPython / dpctl

Changes test_func_order tests for some unary functions #1658