-
Notifications
You must be signed in to change notification settings - Fork 28k
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
[SPARK-47055][PYTHON] Upgrade MyPy 1.8.0 #45115
Conversation
3ebe195
to
35e4d0d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pending CIs.
35e4d0d
to
4be729d
Compare
Ah .. I need to fix MyPy data tests too.. I will turn this to a draft for now. |
4be729d
to
e6c5df5
Compare
if [[ "$MYPY_DATA_TEST" == "true" ]]; then | ||
mypy_data_test | ||
fi | ||
# TODO(SPARK-47057): Reeanble MyPy data test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will enable this in a separate PR once this PR is merged.
Python linter seems to complain still. |
Merged to master. |
…n 3.12 support ### What changes were proposed in this pull request? This PR unpins the version for torch in our CI. This PR is dependent on #45115 ### Why are the changes needed? Testing latest version. This also blocks SPARK-46078. ### Does this PR introduce _any_ user-facing change? No, dev-only. ### How was this patch tested? Manually tested via `./dev/lint-python`. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #45113 from HyukjinKwon/SPARK-47054. Authored-by: Hyukjin Kwon <gurwls223@apache.org> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
### What changes were proposed in this pull request? This is a followup of #45115 that enables MyPy data test. ### Why are the changes needed? To reenable MyPy data test. ### Does this PR introduce _any_ user-facing change? No, test-only. ### How was this patch tested? Manually ran the test via `./dev/lint-python`. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #45135 from HyukjinKwon/SPARK-47057. Authored-by: Hyukjin Kwon <gurwls223@apache.org> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
What changes were proposed in this pull request?
This PR proposes upgrade MyPy to 1.8.0.
Why are the changes needed?
To unblock the full support of Python 3.12 with CI. This unblocks #45113
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manually ran
dev/line-python
Was this patch authored or co-authored using generative AI tooling?
No.