feat(array): only consider arg position, NOT defaultness or keywardness, when binding lambdas in Array.map() and Array.filter()

[NickCrews]

I have this util function:

def is_happy(
    x: ibis.Table | ir.StructValue | None = None,
) -> ir.BooleanValue:
    if x is None:
        x = _
    return ibis.or_(
        x.mood == "HAPPY",
        x.has_eaten_ice_cream,
    )

which I want to be able to use as my_table.filter(is_happy) or my_array_of_structs.filter(is_happy). Before this PR, this fails. Now, it works.

This is a breaking change. Before, this worked: my_array.map(functools.partial(lambda x, y, idx: x + y + idx, y=2)). I don't think this should have ever worked, because this isn't a valid function to call with 2 positional arguments:

>>> import functools
>>> f = functools.partial(lambda x, y, idx: x + y + idx, y=8)
>>> f(1,2)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: <lambda>() got multiple values for argument 'y'

Now, the user MUST supply a function that works as either f(x) or f(x, y) (ie 1 or 2 positional arguments)

This also makes it so that positional-only functions work. Before, def f(x, /) would fail. Now, it works.
This also makes it so that

• func(*args) functions work. We call them as func(elem, idx).
• func(x, *args) functions work. We call them as func(elem, idx).

Basically, the way to explain this is fairly simple, and what I wrote in the docstrings:

           	Callables will be called as `func(element)` or `func(element, idx)`
            depending on if the function accepts 1 or 2+ positional parameters.
            The `idx` argument is the **zero**-based index of each element in the array.

[NickCrews]

This is just a duplicate test of the above test_array_filter_with_index, that's why I'm deleting it.

[github-actions]

ACTION NEEDED

Ibis follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message.

Please update your PR title and description to match the specification.

See https://github.com/ibis-project/ibis/blob/main/.releaserc.js
for the list of acceptable prefixes, eg "feat:", "fix:", "chore:", etc.

The commitlint output is:

⧗   input: feat(array): only use argument position, NOT defaultness or kwargness, when binding lambdas in Array.map() and Array.filter()
✖   header must not be longer than 120 characters, current length is 125 [header-max-length]

✖   found 1 problems, 0 warnings
ⓘ   Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint

[NickCrews]

@cpcloud this is ready for a review whenever you get the chance! I can talk it through with you to if you want. I'd love to get this in as a breaking change in 11.0