Chiara Marmo
Chiara Marmo
Also, please add an entry to the change log at [doc/whats_new/v1.3.rst](https://github.com/scikit-learn/scikit-learn/blob/main/doc/whats_new/v1.3.rst). Like the other entries there, please reference this pull request with `:pr:` and credit yourself (and other contributors if...
@jona-sassenhagen we are good! I am not a core-dev, so you will need to wait for an official approval before merging. Perhaps @jjerphan could find some time? Thanks for your...
Hi @cozek sorry for the late reply and thanks for your pull request. A new error is thrown which should be tested in `test_multiclass.py`. Also other tests are affected by...
Hi @cozek thanks for your patience and your work so far! Waiting for a second approval, do you mind synchronizing with upstream? The continuous integration workflows have changed for version...
I am reopening this one as one of the tasks is still unaddressed: > Regarding the OOB computation: let's just remove it if n_more_estimators == 0 and only do it...
@choo8 , I believe your changes https://github.com/scikit-learn/scikit-learn/pull/24579/files/b2c28778c459d0c58fac1027c618c048543e0764 (with additional tests as specified in https://github.com/scikit-learn/scikit-learn/pull/24579#discussion_r1012729686) will solve this issue. The link with #24764 is important for maintainers to keep in mind...
Closing as fixed by #24532.
Hello... wondering if this PR might be of some interest for the maintainers? Thanks!
Thanks @rjzamora for your review! I'm not sure I am the right person to answer [this comment](https://github.com/dask/dask/pull/10016#discussion_r1229911545) though... :thinking:
Dear maintainers, thank you very much for your comments and detailed reviews. I was planning to come back to this one but I cannot find enough bandwidth to. If some...