Closed twesterhout closed 6 years ago
Hey! Nice work, Tom :) I looked at your code and still found a few errors. I fixed them in a new commit. It now runs through smoothly for me. As for me, this latest commit can be merged into master.
Good catch, Johanna! I've never really tested that code path... shame on me :) Then let's merge.
Here's a suggestion for how to keep working with dictionaries of parameters, but still be able to use LightGBM's sklearn interface. It also runs on lilo4 and lilo5.