-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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] support Python 3.13 #6668
Merged
Merged
Changes from 29 commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
81a2b98
[python-package] support Python 3.13
jameslamb 9069205
Merge branch 'master' into python-3.13
jameslamb fda237c
Merge branch 'master' into python-3.13
jameslamb f3422f1
Merge branch 'master' into python-3.13
jameslamb d9bab62
update everything in the base conda environment, not just conda
jameslamb 9cc314e
python packages now use 'cp' instead of 'cpython'
jameslamb e7256e8
merge master
jameslamb 76f2b8d
fix python_package
jameslamb 3889a78
Merge branch 'master' into python-3.13
jameslamb 9f6cc8b
Merge branch 'master' into python-3.13
jameslamb 82292d3
Merge branch 'master' into python-3.13
jameslamb a797f91
get more logs
jameslamb 612e99c
Merge branch 'master' into python-3.13
jameslamb c1a5a72
try to force a better solver error
jameslamb 9f3d1e5
Merge branch 'master' into python-3.13
jameslamb 6ef9ab6
Merge branch 'master' into python-3.13
jameslamb 89000b3
merge master
jameslamb 47e8dbf
merge latest
jameslamb f471fe0
use build= explicitly
jameslamb 0ed60e4
restore all CI, revert debugging changes
jameslamb e301d3f
Update .ci/test.sh
jameslamb bbec860
empty commit to re-trigger CI
jameslamb 811ee8d
Merge branch 'python-3.13' of github.com:microsoft/LightGBM into pyth…
jameslamb 0db5b2d
update docs conda env
jameslamb 248c076
pin to specific OS and mambaforge
jameslamb 3044b7e
Update .github/workflows/python_package.yml
jameslamb 422be6a
update default Python version in .vsts-ci.yml
jameslamb a78ebbd
remove upper-version pins
jameslamb 30d566f
Merge branch 'master' into python-3.13
jameslamb 6af26b1
update RTD image
jameslamb File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -28,32 +28,32 @@ jobs: | |
include: | ||
- os: macos-13 | ||
task: regular | ||
python_version: '3.10' | ||
python_version: '3.11' | ||
- os: macos-13 | ||
task: sdist | ||
python_version: '3.11' | ||
python_version: '3.12' | ||
- os: macos-13 | ||
task: bdist | ||
python_version: '3.8' | ||
python_version: '3.9' | ||
- os: macos-13 | ||
task: if-else | ||
python_version: '3.9' | ||
python_version: '3.10' | ||
- os: macos-14 | ||
task: bdist | ||
method: wheel | ||
python_version: '3.10' | ||
python_version: '3.11' | ||
- os: macos-13 | ||
task: mpi | ||
method: source | ||
python_version: '3.11' | ||
python_version: '3.12' | ||
- os: macos-13 | ||
task: mpi | ||
method: pip | ||
python_version: '3.12' | ||
python_version: '3.13' | ||
- os: macos-13 | ||
task: mpi | ||
method: wheel | ||
python_version: '3.9' | ||
python_version: '3.10' | ||
steps: | ||
- name: Checkout repository | ||
uses: actions/checkout@v4 | ||
|
@@ -113,12 +113,19 @@ jobs: | |
--rm \ | ||
-v $(pwd):/opt/lgb-build \ | ||
-w /opt/lgb-build \ | ||
python:3.12 \ | ||
python:3.13 \ | ||
/bin/bash ./.ci/test-python-latest.sh | ||
test-oldest-versions: | ||
test-old-versions: | ||
name: Python - oldest supported versions (ubuntu-latest) | ||
runs-on: ubuntu-latest | ||
timeout-minutes: 60 | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
# end-of-life Python versions | ||
python_version: | ||
- '3.7' | ||
- '3.8' | ||
steps: | ||
- name: Checkout repository | ||
uses: actions/checkout@v4 | ||
|
@@ -140,12 +147,12 @@ jobs: | |
--rm \ | ||
-v $(pwd):/opt/lgb-build \ | ||
-w /opt/lgb-build \ | ||
python:3.7 \ | ||
python:${{ matrix.python_version }} \ | ||
/bin/bash ./.ci/test-python-oldest.sh | ||
all-python-package-jobs-successful: | ||
if: always() | ||
runs-on: ubuntu-latest | ||
needs: [test, test-latest-versions, test-oldest-versions] | ||
needs: [test, test-latest-versions, test-old-versions] | ||
steps: | ||
- name: Note that all tests succeeded | ||
uses: re-actors/[email protected] | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Why not to bump version in Appveyor?
LightGBM/.appveyor.yml
Line 6 in 3654eca
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 was thinking that because we use Appveyor to test such an old version of VS Code:
LightGBM/.appveyor.yml
Line 3 in 3654eca
That it should stay frozen on an old Python version too, until supporting that older Python version on Appveyor breaks. Given that, do you think we should make increasing this version to
3.9
part of this update?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.
OK, makes sense! I don't mind leaving 3.8.