Technical debt changes in indexing functions

[ndgrigorian]

This PR cleans up indexing functions

• Renames Python bindings from usm_ndarray_take and usm_ndarray_put to py_take and py_put to improve consistency with the rest of dpctl
• Removes indexing mode macros, opting for a ternary check instead
• Refactors to break up _populate_kernel_params internal function to improve code readability [WIP]

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

[github-actions]

View rendered docs @ https://intelpython.github.io/dpctl/pulls/2012/index.html

[github-actions]

Array API standard conformance tests for dpctl=0.19.0dev0=py310h93fe807_537 ran successfully.
Passed: 894
Failed: 2
Skipped: 118

[github-actions]

Array API standard conformance tests for dpctl=0.19.0dev0=py310h93fe807_537 ran successfully.
Passed: 895
Failed: 1
Skipped: 118

[coveralls]

coverage: 88.233%. remained the same
when pulling 0d7baf0 on technical-debt-indexing-fns
into 7e2b69e on master.