Nicolas Hug

Results 570 comments of Nicolas Hug

But you are creating this new `gradient_original` array right? I thought the point was precisely not to allocate any new array. >you'll have gradients = gradients[partition] right? Maybe that's what...

To be more exact, since `ordered_gradients` is of size `n_samples` and `samples_indices.shape[0]

Not sure there's anything we can do for now, I've opened https://github.com/scipy/scipy/issues/9608

You mean sklearn? LightGBM is doing something very weird with `min_sample_leaf`, it looks like it is ignored because of `num_leaves` (see https://github.com/ogrisel/pygbm/issues/30#issuecomment-435138526)

Hi @VishnuBhaarath , surprise isn't really suited for binary recommendation, you might want to check out https://github.com/benfred/implicit instead

Hi @thabied , sorry I'm really not familiar with AWS but to install surprise you can either install a binary from conda, OR you'll have to build from source (e.g....

Hi @thabied , sorry I'm really not familiar with AWS but to install surprise you can either install a binary from conda, OR you'll have to build from source (e.g....

Thanks for the PR and for the details :) Can you please fix the pep8 issue? >I don't know how to include these measures in GridSearchCV since they require a...

Thanks, I'll leave the PR open and merge it when we find a way to address the previous point. I'd rather not merge it now as it's not fully compatible...

>Is it still due to insufficient memory? Yes most definitely. Jester has about 25k users and only 100 items so the user-based sim takes much more space. It's weird that...