Fix: ufuncs on records should not be allowed unless overridden

[ioanaif]

#1439

[codecov]

Codecov Report

Merging #1559 (55eac8a) into main (78a6535) will increase coverage by 0.26%.
The diff coverage is 75.69%.

Impacted Files	Coverage Δ
src/awkward/_v2/_connect/numpy.py	63.50% <ø> (ø)
src/awkward/_v2/contents/bitmaskedarray.py	66.41% <ø> (-0.76%)	⬇️
src/awkward/_v2/contents/bytemaskedarray.py	88.82% <ø> (-0.28%)	⬇️
src/awkward/_v2/contents/emptyarray.py	71.66% <ø> (ø)
src/awkward/_v2/contents/indexedarray.py	73.83% <ø> (ø)
src/awkward/_v2/contents/listarray.py	91.79% <ø> (ø)
src/awkward/_v2/contents/listoffsetarray.py	81.85% <ø> (ø)
src/awkward/_v2/contents/numpyarray.py	87.34% <ø> (ø)
src/awkward/_v2/contents/regulararray.py	85.48% <ø> (-0.21%)	⬇️
src/awkward/_v2/contents/unionarray.py	86.27% <ø> (ø)
... and 49 more

[ioanaif]

Now that ufuncs are no longer allowed for records in v2, test 0355-mixins fails withError details: cannot broadcast records in absolute @jpivarski

[jpivarski]

While it's possible that we'll find some tests that assume you can apply ufuncs (like np.abs) to generic records and those tests have to be changed or removed, this particular test should still work.

In the definition of Point, a method was defined for np.abs:

awkward/tests/test_0355-mixins.py

Lines 18 to 20 in afe5fc1

	@ak.mixin_class_method(np.abs)
	def point_abs(self):
	return np.sqrt(self.x**2 + self.y**2)

The rule should be that records can't be used in ufuncs unless a method has been defined for that particular ufunc, as is the case here. That's what broadcast_and_apply does; recursively_apply should follow the same rule. (That is, np.add(record, record) should fail if record does not have an implementation of np.add, but it should use that implementation if it does.)

[jpivarski]

Yes! recursively_apply should be just like broadcast_and_apply in requiring behavior and not allowing records unless the records are overloaded.

Just need a test to be sure that np.absolute will work if it is overloaded (like np.add).

Once that test has been added and it passes, go ahead and merge.