ttrouill/complex

Question about concatenation in extract_sub_scores function.

gsp2014 opened this issue · 3 comments

Hi Théo,

I wonder if this line

res = Result(res.preds[idxs], res.true_vals[idxs], res.ranks[np.concatenate((idxs,idxs))], res.raw_ranks[np.concatenate((idxs,idxs))])
should be "res = Result(res.preds[idxs], res.true_vals[idxs], res.ranks[np.concatenate((idxs, len(idxs)+idxs))], res.raw_ranks[np.concatenate((idxs, len(idxs)+idxs))])". That is, add len(idxs) to the concatenation.

Thanks a lot.

Indeed it should be this, will fix asap, thanks!

Actually no! Because idxs is a boolean array that has the same size as res.ranks, and is equal to True at the indexes to extract, so the correct code is indeed np.concatenate((idxs,idxs)).

Sorry for the latency and the wrong first reply :/

Yes, it's my fault. Thanks for your patience!