Antoine Guillaume
Antoine Guillaume
Hey @Datadote, it is considered good practice to assign yourself the issue or use the bot to avoid people working on the same stuff. If you want I could create...
If a line is too long but it is justified, you can add the `#noqa XXX` with XXX being the error code from precommit, the line will be skipped for...
Hi, the goal of the issue would be to update the current file, whether parallelization should happen or not should be set by a `n_jobs` parameter.
Hi @Ramana-Raja , the PR is now merged, do you still intend to work on this ?
I would guess the pre commit issue is coming from your end, other contributors don't have this issue, so I don't know, did you follow the instructions in the documenation...
I don't think we had this issue before, if you find the source we could had it to the documentation if it's relevant. Concerning the original issue, feel free to...
Hi @Sachin-G13 thanks for the PR, I think you can make the following adjustments : - The description of the deep learning base class does not bring much here, I...
@Sachin-G13 Are you still intending to work on this ?
Something coming up in my mind after review, did we handle the multiclass case in #2017 ? Following the same logic as the binary case, we would need to do...
Think so ? In multi class the linear methods learn one set of coefficient for each class. (`n_classes, n_channels`) instead of `n_channels` in binary case