-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
CI: move Linux and MacOS Azure builds to conda lock files #22448
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
Merged
Merged
Changes from all commits
Commits
Show all changes
115 commits
Select commit
Hold shift + click to select a range
36279ff
First attempt at using lock files in the CI
lesteve dafedd6
fix
lesteve cc3d965
debug
lesteve 1a08f1a
it
lesteve 6a74cb1
fix
lesteve c2ae825
fix
lesteve a23dc48
fix
lesteve 6a47b49
Need openmp in OSX build
lesteve e53f6fa
Merge branch 'main' into ci-lock-file
lesteve d2a6dea
Use lock file in py37_conda_defaults_openblas build.
lesteve 86638b8
add missing lock and env file
lesteve e890a4b
fix lock filename
lesteve 854c735
Switch to conda-lock dev version + regenerate lock files
lesteve 08fc05a
Fix lock filenames
lesteve 4a41fdf
Using conda-lock.yml at the end of the file seems to make conda-lock …
lesteve 9b793d5
debug with tmate
lesteve c987848
Try conda update
lesteve d0e9115
Fix macOS
lesteve ab2337e
debug
lesteve 09fb44d
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve 2531ea6
Re-add logic for LOCK_FILE
lesteve aacd4a3
iter
lesteve 988c71b
Try to use pip-compile for ubuntu_atlas
lesteve efbc928
fix
lesteve 34153a7
fix
lesteve 4004552
no coverage
lesteve 7365339
Use correct joblib oldest supported version
lesteve 4b8e8e7
Add generation of ubuntu_atlas to generate script
lesteve 2864b25
Use latest conda-lock version which includes fixes for matplotlib pip…
lesteve d73707a
Use lock file for latest versions with pip
lesteve 4890bb6
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve f19b87b
Move to python 3.8 and more recent min dependencies
lesteve 71653a1
Update script + regenerate lock files
lesteve 9643017
Use lock file for pylatest_pip_openblas_pandas build
lesteve 71a669b
fix
lesteve f0f6027
use mamba that seems to have a consistent order for lock files if we …
lesteve 88e18c1
Use python from defaults to mirror what is done in main
lesteve 7daef3a
Move py38_conda_forge_openblas_ubuntu_1804 to lock file
lesteve 2026d4c
Add lock file for pylatest_conda_mkl_no_openmp build
lesteve 8f8b160
Add windows py38_conda_forge_mkl build
lesteve 60c1d5f
fix
lesteve 4625ce6
fix
lesteve 2466b9c
debug
lesteve 96af7b1
debug
lesteve 0f2eb46
revert windows stuff
lesteve 85f2204
Use conda-lock release
lesteve fa99f80
cosmetics
lesteve 76bac7d
update script to work from root folder
lesteve 688524b
Move debian-32 buid to lock file
lesteve 394a9b8
debug
lesteve e88d718
debug
lesteve 4e8313e
debug
lesteve d706ab3
pass env var to docker
lesteve 62a4563
Need PYTEST_XDIST_VERSION at test-time ...
lesteve 2baa2e7
Remove dead code
lesteve b72ea80
[icc-build] move icc build to lock file + tweaks
lesteve 977da49
[icc-build] [pypy] fixes
lesteve 89f9c26
[icc-build] [pypy] fixes
lesteve 47858f9
[icc-build] [pypy] fixes
lesteve f737891
[pypy] use normal python in base environment
lesteve 620fd91
[icc-build] [pypy] Remove more dead code
lesteve 7dd24c0
[scipy-dev] Hacky way to support scipy-dev build to work-around conda…
lesteve e3452fb
not using mamba anymore
lesteve 9f9fec0
[scipy-dev] trigger nightly build
lesteve a14ba48
[scipy-dev] fix lockfile
lesteve 87a0824
[scipy-dev] use defaults channel to mirror main
lesteve b535d24
[scipy-dev] trigger nightly build
lesteve 1bc7a6d
[scipy-dev] debug
lesteve a3b9269
[scipy-dev] pin pytest
lesteve 10fb4cb
remove debug
lesteve b0f144f
Remove unused variables
lesteve 54bbb28
Merge branch 'main' into ci-lock-file
lesteve 566657c
remove unneeded line
lesteve d86843c
Add environment + script to generate them
lesteve a81a793
Regenerate lock files from environment/requirements file
lesteve 324f8fd
rename script to generate environment file
lesteve 55f7533
pin conda-lock
lesteve 94bfc22
Move environment + lock file updates into a single script.
lesteve 0269fc6
[scipy-dev] [pypy] [icc-build] Trigger CI
lesteve ceedbb3
Add docstring to script + other tweaks
lesteve cea7597
cosmit
lesteve 06e749e
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve abec451
Apply suggestions from code review
lesteve d5217fd
Update script + regenerate lock files
lesteve b9be31e
Switch to explicit lock files
lesteve d3d6087
Fix
lesteve 08fe15a
Fix lockfile names
lesteve 4cc177e
Needs conda-lock for conda+pip envs
lesteve 9fcd718
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve d5c44fe
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve f720d49
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve b1f0050
Update build_tools/azure/update_environments_and_lock_files.py
lesteve 1afb586
Install alpha Cython version from PyPI
lesteve 4d894e2
[scipy-dev] Remove numpy et al from environment.yml in scipy-dev build.
lesteve 4f641b4
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve 2fd8850
[scipy-dev] [pypy] [icc-build] Trigger CI
lesteve ff4f403
Add comments about script dependencies
lesteve 40b1953
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve 17c036c
Trigger CI
lesteve 33e1b4a
[scipy-dev] [pypy] [icc-build] Trigger CI
lesteve e24be39
Add text about default_package_constraints
lesteve a8b2a7e
Make script run on osx-arm64 machines
lesteve f96b00e
Update environment and lock files
lesteve 6fc77e5
[scipy-dev] [pypy] [icc-build] Trigger CI
lesteve e56334a
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve 8cf6d64
[scipy-dev] [pypy] [icc-build] [nogil] Trigger CI
lesteve c5e8be9
Update lock files
lesteve 4edb574
wip
lesteve 054a0d3
[scipy-dev] [pypy] [icc-build] [nogil] Add lock files for python-nogil
lesteve 7c4f4a1
[scipy-dev] [pypy] [icc-build] [nogil] Fix
lesteve fb1d21f
Remove nogil build from script
lesteve d8c11e4
Use --upgrade to for pip-compile to update dependencies
lesteve b81fea9
Rerun script
lesteve 2f4ebdc
Move comment to requirement file + add --upgrade
lesteve 66f0f07
Merge branch 'main' of https://github.com/scikit-learn/scikit-learn i…
lesteve 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 hidden or 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 hidden or 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 |
---|---|---|
@@ -0,0 +1,34 @@ | ||
# | ||
# This file is autogenerated by pip-compile with python 3.9 | ||
# To update, run: | ||
# | ||
# pip-compile --output-file=build_tools/azure/debian_atlas_32bit_lock.txt build_tools/azure/debian_atlas_32bit_requirements.txt | ||
# | ||
atomicwrites==1.4.0 | ||
# via pytest | ||
attrs==21.4.0 | ||
# via pytest | ||
cython==0.29.28 | ||
# via -r build_tools/azure/debian_atlas_32bit_requirements.txt | ||
importlib-metadata==4.11.3 | ||
# via pytest | ||
joblib==1.0.0 | ||
# via -r build_tools/azure/debian_atlas_32bit_requirements.txt | ||
more-itertools==8.13.0 | ||
# via pytest | ||
packaging==21.3 | ||
# via pytest | ||
pluggy==0.13.1 | ||
# via pytest | ||
py==1.11.0 | ||
# via pytest | ||
pyparsing==3.0.9 | ||
# via packaging | ||
pytest==5.0.1 | ||
# via -r build_tools/azure/debian_atlas_32bit_requirements.txt | ||
threadpoolctl==2.2.0 | ||
# via -r build_tools/azure/debian_atlas_32bit_requirements.txt | ||
wcwidth==0.2.5 | ||
# via pytest | ||
zipp==3.8.0 | ||
# via importlib-metadata |
This file contains hidden or 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 |
---|---|---|
@@ -0,0 +1,7 @@ | ||
# DO NOT EDIT: this file is generated from the specification found in the | ||
# following script to centralize the configuration for all Azure CI builds: | ||
# build_tools/azure/update_environments_and_lock_files.py | ||
cython | ||
joblib==1.0.0 # min | ||
threadpoolctl==2.2.0 | ||
pytest==5.0.1 # min |
This file contains hidden or 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.
Can we remove
PYTEST_XDIST_VERSION
as well? I do not think it is being used anymore.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.
PYTEST_XDIST_VERSION
is still used in the test script:scikit-learn/build_tools/azure/test_script.sh
Lines 65 to 66 in 7bdd426
Longer term, I was thinking about cleaning up environment variables like
PYTEST_XDIST_VERSION
andCOVERAGE
and check whether a package is installed instead of relying on a environment variable.