MNT fix linter issues related to multioutput #24143
Conversation
|
I didn't review line by line but since the linter is green now, this should be good to merge. |
|
I just ran isort on the files, no manual change. |
isort of expected that (okay, I'll see myself out) |
There was a problem hiding this comment.
Running isort makes it harder to see what this actually changed. I was able to figure it out only by looking at the linting error in the other PR
There is also strange ordering with the "Authors" section and the abc import. I prefer fixing the linting issues directly without isort.
sklearn/multioutput.py
Outdated
| from abc import ABCMeta, abstractmethod | ||
|
|
There was a problem hiding this comment.
This is being imported before the # Authors section. I'm guessing isort needs an empty line above from numbers import Integral to function correctly.
|
Merge this one? |
Fixes linter issues related to multioutput which were not detected by the CI, or were mistakes by my merge from
main.cc @thomasjpfan @BenjaminBossan