Skip to content

Commit 17859ed

Browse files
sdpythonsdpython
authored
Merge pull request #118 from sdpython/ref
Refactoring, ruff, black, cmake
2 parents 4b610ba + ee563d3 commit 17859ed

File tree

268 files changed

+13558
-20982
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

268 files changed

+13558
-20982
lines changed

‎.circleci/config.yml‎

Lines changed: 0 additions & 100 deletions
This file was deleted.

‎.github/workflows/black-ruff.yml‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
name: Black + Ruff Format Checker
2+
on: [push, pull_request]
3+
jobs:
4+
black-format-check:
5+
runs-on: ubuntu-latest
6+
steps:
7+
- uses: actions/checkout@v2
8+
- uses: psf/black@stable
9+
with:
10+
options: "--diff --check"
11+
src: "."
12+
ruff-format-check:
13+
runs-on: ubuntu-latest
14+
steps:
15+
- uses: actions/checkout@v3
16+
- uses: chartboost/ruff-action@v1

‎.github/workflows/check-urls.yml‎

Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,47 @@
1+
name: Check URLs
2+
3+
on:
4+
pull_request:
5+
branches: [main]
6+
schedule:
7+
# ┌───────────── minute (0 - 59)
8+
# │ ┌───────────── hour (0 - 23)
9+
# │ │ ┌───────────── day of the month (1 - 31)
10+
# │ │ │ ┌───────────── month (1 - 12 or JAN-DEC)
11+
# │ │ │ │ ┌───────────── day of the week (0 - 6 or SUN-SAT)
12+
# │ │ │ │ │
13+
# │ │ │ │ │
14+
# │ │ │ │ │
15+
# * * * * *
16+
- cron: '30 1 * * 0'
17+
18+
jobs:
19+
build:
20+
runs-on: ubuntu-latest
21+
22+
steps:
23+
- uses: actions/checkout@v3
24+
25+
- name: urls-checker-code
26+
uses: urlstechie/urlchecker-action@master
27+
with:
28+
subfolder: mlinsights
29+
file_types: .md,.py,.rst,.ipynb
30+
print_all: false
31+
timeout: 2
32+
retry_count# : 2
33+
exclude_urls: https://github.com/microsoft/onnxruntime/blob/
34+
exclude_patterns: https://github.com/microsoft/onnxruntime/blob/
35+
# force_pass : true
36+
37+
- name: urls-checker-docs
38+
uses: urlstechie/urlchecker-action@master
39+
with:
40+
subfolder: _doc
41+
file_types: .md,.py,.rst,.ipynb
42+
print_all: false
43+
timeout: 2
44+
retry_count# : 2
45+
exclude_urls: 64,14: https://github.com/Kitware/CMake/releases/download/v${cmake_version}/cmake-$,https://developer.download.nvidia.com/compute/cuda/$
46+
exclude_patterns: https://www.data.gouv.fr/fr/datasets/r/e3d83ab3-dc52-4c99-abaf-8a38050cc68c,https://dev.azure.com/
47+
# force_pass : true

‎.github/workflows/clang.yml‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
name: Clang Format Checker
2+
on: [push]
3+
jobs:
4+
clang-format-checking:
5+
runs-on: ubuntu-latest
6+
steps:
7+
- uses: actions/checkout@v2
8+
- uses: RafikFarhad/clang-format-github-action@v3
9+
with:
10+
sources: "src/**/*.h,src/**/*.c,test/**/*.c"

‎.github/workflows/cmakelint.yml‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
name: Cmake Format Checker
2+
3+
on: [push]
4+
5+
jobs:
6+
build:
7+
runs-on: ubuntu-latest
8+
9+
steps:
10+
- name: Checkout repository
11+
uses: actions/checkout@v2
12+
13+
- name: Format CMake files
14+
id: cmake-format
15+
uses: PuneetMatharu/[email protected]
16+
with:
17+
args: --check
18+
19+
- name: Commit changes
20+
uses: stefanzweifel/git-auto-commit-action@v4
21+
with:
22+
commit_user_name: cmake-format-bot
23+
commit_message: 'Automated commit of cmake-format changes.'

‎.github/workflows/documentation.yml‎

Lines changed: 91 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,91 @@
1+
name: Documentation and Code Coverage
2+
3+
on:
4+
push:
5+
pull_request:
6+
types:
7+
- closed
8+
branches:
9+
- main
10+
11+
jobs:
12+
run:
13+
name: Build documentation on ${{ matrix.os }}
14+
runs-on: ${{ matrix.os }}
15+
strategy:
16+
matrix:
17+
os: [ubuntu-latest]
18+
19+
steps:
20+
- uses: actions/checkout@v3
21+
22+
- uses: actions/setup-python@v4
23+
with:
24+
python-version: '3.11'
25+
26+
- uses: tlylt/install-graphviz@v1
27+
28+
- name: Install pandoc
29+
run: sudo apt-get install -y pandoc
30+
31+
- name: Install requirements
32+
run: python -m pip install -r requirements.txt
33+
34+
- name: Install requirements dev
35+
run: python -m pip install -r requirements-dev.txt
36+
37+
- name: Cache pip
38+
uses: actions/cache@v2
39+
with:
40+
path: ~/.cache/pip
41+
key: ${{ runner.os }}-pip-${{ hashFiles('requirements-dev.txt') }}
42+
restore-keys: |
43+
${{ runner.os }}-pip-
44+
${{ runner.os }}-
45+
46+
- name: Build
47+
run: python setup.py build_ext --inplace
48+
49+
- name: Generate coverage report
50+
run: |
51+
pip install pytest
52+
pip install pytest-cov
53+
export PYTHONPATH=.
54+
pytest --cov=./mlinsights/ --cov-report=xml --durations=10 --ignore-glob=**LONG*.py --ignore-glob=**notebook*.py
55+
export PYTHONPATH=
56+
57+
- name: Upload coverage reports to Codecov
58+
uses: codecov/codecov-action@v3
59+
env:
60+
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
61+
62+
- name: Install
63+
run: python setup.py install
64+
65+
- name: Copy license, changelogs
66+
run: |
67+
cp LICENSE* ./_doc
68+
cp CHANGELOGS* ./_doc
69+
70+
- name: Documentation
71+
run: python -m sphinx ./_doc ./dist/html -n -w doc.txt
72+
73+
- name: Summary
74+
run: cat doc.txt
75+
76+
- name: Check for errors and warnings
77+
run: |
78+
if [[ $(grep ERROR doc.txt | grep -v 'validation.cuda') ]]; then
79+
echo "Documentation produces errors."
80+
grep ERROR doc.txt | grep -v 'validation.cuda'
81+
exit 1
82+
fi
83+
if [[ $(grep WARNING doc.txt | grep -v 'validation.cuda') ]]; then
84+
echo "Documentation produces warnings."
85+
grep WARNING doc.txt | grep -v 'validation.cuda'
86+
exit 1
87+
fi
88+
89+
- uses: actions/upload-artifact@v3
90+
with:
91+
path: ./dist/html/**

‎.github/workflows/rstcheck.yml‎

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
name: RST Check
2+
3+
on: [push, pull_request]
4+
5+
jobs:
6+
build_wheels:
7+
name: rstcheck ${{ matrix.os }}
8+
runs-on: ${{ matrix.os }}
9+
strategy:
10+
matrix:
11+
os: [ubuntu-latest]
12+
13+
steps:
14+
- uses: actions/checkout@v3
15+
16+
# Used to host cibuildwheel
17+
- uses: actions/setup-python@v4
18+
with:
19+
python-version: '3.11'
20+
21+
- name: Install requirements
22+
run: python -m pip install -r requirements.txt
23+
24+
- name: Install rstcheck
25+
run: python -m pip install sphinx tomli rstcheck[toml,sphinx]
26+
27+
- name: rstcheck
28+
run: rstcheck -r _doc mlinsights

‎.github/workflows/wheels-linux.yml‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
name: Build Wheel Linux
2+
3+
on:
4+
push:
5+
branches:
6+
- main
7+
- 'releases/**'
8+
9+
jobs:
10+
build_wheels:
11+
name: Build wheels on ${{ matrix.os }}
12+
runs-on: ${{ matrix.os }}
13+
strategy:
14+
matrix:
15+
os: [ubuntu-latest]
16+
17+
steps:
18+
- uses: actions/checkout@v3
19+
20+
# Used to host cibuildwheel
21+
- uses: actions/setup-python@v4
22+
with:
23+
python-version: '3.10'
24+
25+
- name: Install cibuildwheel
26+
run: python -m pip install cibuildwheel
27+
28+
- name: python version
29+
run: python -V
30+
31+
- name: Build wheels
32+
run: python -m cibuildwheel --output-dir wheelhouse
33+
# to supply options, put them in 'env', like:
34+
#env:
35+
# # CIBW_BUILD: "cp310* cp311*"
36+
# CIBW_SKIP: cp36-* cp37-* cp38-* cp39-*
37+
38+
- uses: actions/upload-artifact@v3
39+
with:
40+
path: ./wheelhouse/*.whl

0 commit comments

Comments
 (0)