-
Notifications
You must be signed in to change notification settings - Fork 13
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'master' of https://github.com/sdpython/mlinsights into dev
- Loading branch information
Showing
50 changed files
with
1,080 additions
and
305 deletions.
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,27 +1,37 @@ | ||
dist: bionic | ||
dist: focal | ||
sudo: true | ||
language: python | ||
|
||
matrix: | ||
include: | ||
- python: 3.7 | ||
name: "Py37-skl022" | ||
env: sklearnc="==0.22.2" | ||
- python: 3.9 | ||
name: "Py39-024+" | ||
env: | ||
- sklver=">=0.24.0" | ||
- jlver=">=1.0" | ||
- python: 3.8 | ||
name: "Py38-skl023" | ||
env: sklearnc=">=0.23" | ||
name: "Py38-023" | ||
env: | ||
- sklver="==0.23.2" | ||
- jlver="==0.17.0" | ||
|
||
before_install: | ||
- sudo add-apt-repository -y ppa:ubuntu-toolchain-r/test | ||
- sudo apt-get update -qq | ||
- sudo apt-get install graphviz | ||
- gcc --version | ||
- sudo apt-get install libgeos-dev libproj-dev proj-data graphviz libblas-dev liblapack-dev | ||
- wget https://apt.llvm.org/llvm.sh | ||
- chmod +x llvm.sh | ||
- sudo ./llvm.sh 10 | ||
- ls /usr/bin/llvm* | ||
- export LLVM_CONFIG=/usr/bin/llvm-config | ||
# - sudo ln -s /usr/bin/llvm-config-10 /usr/bin/llvm-config | ||
- sudo apt-get -y install graphviz | ||
|
||
install: | ||
- pip install pyquickhelper cython scipy | ||
- pip install -r requirements.txt | ||
- pip install -r requirements-ext.txt | ||
- pip install scikit-learn$sklearnc | ||
- python -c "import sklearn;print(sklearn.__version__)" | ||
- pip install "scikit-learn$sklver" | ||
- pip install "joblib$jlver" | ||
|
||
before_script: | ||
- python -u setup.py build_ext --inplace --verbose | ||
- python setup.py build_ext --inplace | ||
|
||
script: | ||
- python setup.py unittests |
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
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 |
---|---|---|
@@ -0,0 +1,22 @@ | ||
|
||
.. blogpost:: | ||
:title: scikit-learn internal API | ||
:keywords: API | ||
:date: 2020-09-02 | ||
:categories: scikit-learn | ||
:lid: blog-internal-api-impurity-improvement | ||
|
||
The signature of method `impurity_improvement | ||
<https://github.com/scikit-learn/scikit-learn/blob/master/ | ||
sklearn/tree/_criterion.pxd#L65>`_ will change for version | ||
0.24. That's usually easy to handle two versions of scikit-learn | ||
even overloaded in a class except that method is implemented | ||
in :epkg:`cython`. The method must be overloaded the same way | ||
with the same signature. The way it was handled is implemented | ||
in PR `88 <https://github.com/sdpython/mlinsights/pull/88>`_. | ||
|
||
The best would be to include both of them but only one of | ||
them can compile. I did not find any good solution to that. | ||
It compiles whatever scikit-learn's version but the compiled | ||
module only works with the installed version of | ||
:epkg:`scikti-learn`. |
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 |
---|---|---|
@@ -0,0 +1,15 @@ | ||
|
||
.. blogpost:: | ||
:title: scikit-learn 0.23 | ||
:keywords: scikit-learn, 0.23, 0.24 | ||
:date: 2021-01-03 | ||
:categories: scikit-learn | ||
|
||
The unit test are run against | ||
:epkg:`scikit-learn` 0.23, 0.24. | ||
Some unit tests are failing with version 0.23. | ||
They were disabled instead of looking into a cause | ||
which does not appear with the latest version. | ||
It affects all classes inheriting from :class:`SkBase | ||
<mlinsights.sklapi.sklearn_base.SkBase>` where a model | ||
using it is trained. The issue happens in :epkg:`joblib`. |
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.