-
Notifications
You must be signed in to change notification settings - Fork 3.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[python-package] fix mypy error about missing type hint in dask.py #4840
Conversation
I think the first time LightGBM/python-package/lightgbm/dask.py Line 945 in b94577d
List[List[dask_Array]] , where each entry has as many items as there are partitions. Then the contents get overwritten in LightGBM/python-package/lightgbm/dask.py Line 959 in b94577d
List[dask_Array] .
I think variables changing types is a big no-no for |
ooooooo you're right! I missed that
You're generally right about |
just re-ran the R-package jobs, let's see if the MiKTeX issue (#4005 (comment)) was fixed overnight |
🎉 CI seems happy again! e.g. from this Windows build
|
This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this. |
Contributes to #3867.
This PR fixes one
mypy
error indask.py
.Confirmed that this suppresses these warnings by running the following.
mypy \ --exclude='python-package/compile/|python-package/build' \ --ignore-missing-imports \ python-package/