Open dntaylor opened 4 years ago
Thanks for pointing out this bug!
It does look easy enough to fix, but at the same time, it would be a good idea to consider moving to Awkward 1 (https://github.com/scikit-hep/awkward-1.0). The string implementation is considerably improved and you can use both in the same script because the package names are different. (In particular, I remember explicitly dealing with the case of broadcasting a string as a scalar to all elements in an array of strings—I doubt the old one would do it right, even with this fix.)
I'll be advertising the transition more after Uproot supports the new one, but it's usable not with ak.from_awkward0 and ak.to_awkward0.
There is a bug in StringArray when comparing strings:
I guess it needs either enumerate or to remove the "i" (since it isn't used).