Skip to content

Mark sklearn tests #1202

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.

Sign up for GitHub

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
mfeurer merged 15 commits into from
Feb 23, 2023
Merged

Mark sklearn tests #1202

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
21cd8d7
Add sklearn marker
PGijsbers Feb 12, 2023
e2dcdc2
Mark tests that use scikit-learn
PGijsbers Feb 12, 2023
9ad6dae
Only run scikit-learn tests multiple times
PGijsbers Feb 12, 2023
d67b7cc
Rename for correct variable
PGijsbers Feb 20, 2023
f72ff8b
Add sklearn mark for filesystem test
PGijsbers Feb 20, 2023
e89db28
Remove quotes around sklearn
PGijsbers Feb 21, 2023
b547589
Instead include sklearn in the matrix definition
PGijsbers Feb 22, 2023
184c89c
Update jobnames
PGijsbers Feb 22, 2023
7a2a850
Add explicit false to jobname
PGijsbers Feb 22, 2023
657f0d3
Remove space
PGijsbers Feb 22, 2023
f36a040
Add function inside of expression?
PGijsbers Feb 22, 2023
ff34515
Do string testing instead
PGijsbers Feb 22, 2023
e549aa9
Add missing ${{
PGijsbers Feb 22, 2023
1db1ff7
Add explicit true to old sklearn tests
PGijsbers Feb 22, 2023
d01b4b2
Add instruction to add pytest marker for sklearn tests
PGijsbers Feb 23, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Only run scikit-learn tests multiple times 
The generic tests that don't use scikit-learn should only be tested once
(per platform).
  • Loading branch information
@PGijsbers
PGijsbers committed Feb 23, 2023
commit 9ad6daeac503bfa486bb9f693ec0793c063f0a9c
5 changes: 4 additions & 1 deletion .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ jobs:
scikit-learn: 0.23.1
code-cov: true
os: ubuntu-latest
all-tests: true
- os: windows-latest
scikit-learn: 0.24.*
fail-fast: false
Expand Down Expand Up @@ -62,7 +63,9 @@ jobs:
if: matrix.os == 'ubuntu-latest'
run: |
if [ ${{ matrix.code-cov }} ]; then codecov='--cov=openml --long --cov-report=xml'; fi
pytest -n 4 --durations=20 --timeout=600 --timeout-method=thread --dist load -sv $codecov --reruns 5 --reruns-delay 1
# Most of the time, running only the scikit-learn tests is sufficient
if [ ! ${{ matrix.all-tests }} ]; then sklearn='-m "sklearn"'; fi
pytest -n 4 --durations=20 --timeout=600 --timeout-method=thread --dist load -sv $codecov $sklearn_select --reruns 5 --reruns-delay 1
Comment thread
PGijsbers marked this conversation as resolved.
Outdated
- name: Run tests on Windows
if: matrix.os == 'windows-latest'
run: | # we need a separate step because of the bash-specific if-statement in the previous one.
Expand Down