-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
BLD Migrate away from distutils and only use setuptools #24563
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
Changes from all commits
771573e
9ba6a47
68f84a1
c524327
0c510bf
739c11a
d0f4a7a
ba8a018
979d764
7baf7d0
0efa8b1
5dda5d4
429eed6
2f547bc
bd22900
af0b25c
bc55cd7
aebf345
2d162e8
a4db54e
8a12d9b
38b61b5
dd60027
4c18f92
24ac0fb
a9feb59
8661809
aad2bff
ee3c884
98fe1e2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -106,27 +106,6 @@ jobs: | |||||||||||||
LOCK_FILE: './build_tools/azure/python_nogil_lock.txt' | ||||||||||||||
COVERAGE: 'false' | ||||||||||||||
|
||||||||||||||
# Check compilation with intel C++ compiler (ICC) | ||||||||||||||
- template: build_tools/azure/posix.yml | ||||||||||||||
parameters: | ||||||||||||||
name: Linux_Nightly_ICC | ||||||||||||||
vmImage: ubuntu-20.04 | ||||||||||||||
dependsOn: [git_commit, linting] | ||||||||||||||
condition: | | ||||||||||||||
and( | ||||||||||||||
succeeded(), | ||||||||||||||
not(contains(dependencies['git_commit']['outputs']['commit.message'], '[ci skip]')), | ||||||||||||||
or(eq(variables['Build.Reason'], 'Schedule'), | ||||||||||||||
contains(dependencies['git_commit']['outputs']['commit.message'], '[icc-build]') | ||||||||||||||
) | ||||||||||||||
) | ||||||||||||||
matrix: | ||||||||||||||
pylatest_conda_forge_mkl: | ||||||||||||||
DISTRIB: 'conda' | ||||||||||||||
LOCK_FILE: 'build_tools/azure/pylatest_conda_forge_mkl_no_coverage_linux-64_conda.lock' | ||||||||||||||
COVERAGE: 'false' | ||||||||||||||
BUILD_WITH_ICC: 'true' | ||||||||||||||
|
||||||||||||||
Comment on lines
-109
to
-129
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There are a few occurrences of ICC in the code-base, namely: scikit-learn/doc/developers/contributing.rst Line 552 in f905c21
scikit-learn/sklearn/_build_utils/openmp_helpers.py Lines 21 to 25 in bc55cd7
Should we let users be able to use icc? or should we remove those if we do not test the support of this compiler on the CI? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think we should keep this, assuming setting the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I did not get the Intel compilers to work by setting the (The Azure logs does not exist anymore because it has been too long) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ok let's keep that in mind for a follow-up PR. |
||||||||||||||
- template: build_tools/azure/posix-docker.yml | ||||||||||||||
parameters: | ||||||||||||||
name: Linux_Nightly_PyPy | ||||||||||||||
|
@@ -182,7 +161,6 @@ jobs: | F438|||||||||||||
DISTRIB: 'conda' | ||||||||||||||
LOCK_FILE: './build_tools/azure/py38_conda_forge_openblas_ubuntu_2204_linux-64_conda.lock' | ||||||||||||||
COVERAGE: 'false' | ||||||||||||||
BUILD_WITH_ICC: 'false' | ||||||||||||||
SKLEARN_TESTS_GLOBAL_RANDOM_SEED: '0' # non-default seed | ||||||||||||||
|
||||||||||||||
- template: build_tools/azure/posix.yml | ||||||||||||||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,9 @@ | ||
[options] | ||
packages = find_namespace: | ||
|
||
[options.packages.find] | ||
include = sklearn* | ||
|
||
[aliases] | ||
test = pytest | ||
|
||
|
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.
This is required for the
sdist
to contain*.cpp
files that is not a source file. Specifically,svm/src/libsvm/svm.cpp
.