oneDAL icon indicating copy to clipboard operation
oneDAL copied to clipboard

Update oneapi random forest oob

Open vlad-nazarov opened this issue 2 years ago • 1 comments

vlad-nazarov avatar Dec 07 '21 21:12 vlad-nazarov

/intelci: run

vlad-nazarov avatar Dec 08 '21 08:12 vlad-nazarov

@Alexsandruss - you fixed something related to oob score just a few days ago. Can you please check if this PR is somewhat important in this regard or no?

KulikovNikita avatar Mar 21 '23 11:03 KulikovNikita

This specifically modifies some files which I am not sure are actually used. I'm going to look into it.

I believe that the oneDAL implementation may supersede the DAAL defined TrainBatchKernelOneAPI versions.

icfaust avatar Mar 23 '23 08:03 icfaust

This code change should be accepted. However, this only impacts running DAAL not on CPUs. We should set a clear standard about future support.

icfaust avatar Mar 23 '23 09:03 icfaust

Change no longer necessary.

icfaust avatar Mar 23 '23 09:03 icfaust