bool keys fail to target bool axes, even when explicitly specified

[gdementen]

>>> arr = LArray([0.1, 0.2], Axis([False, True], 'gender'))
>>> key = np.array([True, False, True, True])
>>> arr[key]
ValueError: boolean key with a different shape ((4,)) than array ((2,))
>>> arr.points[key]
ValueError: boolean key with a different shape ((4,)) than array ((2,))

Oddly, using list keys seems to work (as long as we only target a single dimension)

>>> arr[[True, False, True, True]]
gender  True  False  True  True
         0.2    0.1   0.2   0.2
>>> arr.points[[True, False, True, True]]
gender  True  False  True  True
         0.2    0.1   0.2   0.2

Using an LArray key does not seem to help either:

>>> arr[LArray(key, Axis(6, 'gender'))]
ValueError: gender[0, 2, 3] is not a valid label for any axis

This is a "regression" in 0.29. It is only a partial regression because it was known to be broken before. It's just more broken now. 😄 See liam2/liam2#296.

[gdementen]

tentatively adding to 0.30. Will push to 0.31 if necessary, but this should be done sooner rather than later.

[benjello]

@gdementen : when do you think you could look into this. I am stucked in a dependence nightmare. I want to compile liam2/master in a Python3.7. liam2 requires larray=0.28 which requires older version of pandas which cannot be installed Cython/numpy problems. I would like to use the latest version of pandas.

[gdementen]

Ouch. Sorry about this mess, but this will take a while. Certainly not before mid march and probably not before the end of may and even that I cannot promise because LIAM2 is not the only project I had to put aside for a while. FWIW, I am spending most of my time these days on the website which will hold the results of the evaluation of political parties programs that the FPB is doing for the belgian elections in May. I am truly sorry about this situation but I cannot help it.

If you are desperate, you could use larray 0.28 with a backport of ca21828 to add support for Pandas >= 0.21.

[benjello]

I am desperate. Thanks for the hint !

[benjello]

It worked :-)
Could I push a branch called ipp on this repo for this backport ? Since I need to ask people from a secured data center to download it for me I would prefer to do the cheery-pick here than there. The fact that it is the official repo may also make it feel safer.

[gdementen]

Sure.