mlfinlab icon indicating copy to clipboard operation
mlfinlab copied to clipboard

get_signal(prob, num_classes, pred=None) maybe wrong.

Open qtbgo opened this issue 3 years ago • 0 comments

Hi, in your implementation of get_signal function (book snippet 10.1), you say : Note 1: In the book, this function contains a conditional statement checking for a column named 'side', then executes what is essentially the above line. This has been removed as it appears to be redundant and simplifies the function I don't think your modification is correct. In fact, parameter "pred" should be explained as predicted label x, not "side".

qtbgo avatar Oct 30 '21 03:10 qtbgo