Skip to content

Commit

Permalink
fix build CI failing due to old pip-tools failing with latest pip (#2200
Browse files Browse the repository at this point in the history
)
  • Loading branch information
imatiach-msft authored Aug 2, 2023
1 parent 06c00b3 commit 0327bb2
Show file tree
Hide file tree
Showing 9 changed files with 31 additions and 13 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/CI-e2e-notebooks-vision.yml
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ jobs:
run: |
python -m pip install --upgrade pip
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=6.13.0"
pip install --upgrade "pip-tools<=7.1.0"
- name: Install dependencies
shell: bash -l {0}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/CI-e2e-notebooks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ jobs:
run: |
python -m pip install --upgrade pip
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=6.13.0"
pip install --upgrade "pip-tools<=7.1.0"
- name: Install dependencies
shell: bash -l {0}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/CI-notebook-vision.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ jobs:
run: |
python -m pip install --upgrade pip
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=6.13.0"
pip install --upgrade "pip-tools<=7.1.0"
- name: Install dependencies
shell: bash -l {0}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/CI-notebook.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ jobs:
run: |
python -m pip install --upgrade pip
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=6.13.0"
pip install --upgrade "pip-tools<=7.1.0"
- name: Install dependencies
shell: bash -l {0}
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/CI-python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,20 @@ jobs:
with:
python-version: ${{ matrix.pythonVersion }}

- name: Setup tools
- if: ${{ matrix.pythonVersion == '3.6' || matrix.pythonVersion == '3.7' }}
name: Setup tools for python lt 3.8
run: |
python -m pip install --upgrade pip
python -m pip install --upgrade "pip<=23.1.2"
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=6.13.0"
- if: ${{ matrix.pythonVersion == '3.8' || matrix.pythonVersion == '3.9' || matrix.pythonVersion == '3.10' }}
name: Setup tools for python gte 3.8
run: |
python -m pip install --upgrade pip
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=7.1.0"
- name: Pip compile
run: |
pip-compile requirements-dev.txt -o requirements-dev-comp.txt
Expand Down
14 changes: 11 additions & 3 deletions .github/workflows/CI-rai_core_flask-pytest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ on:
- ".github/workflows/CI-rai_core_flask-pytest.yml"

jobs:
ci-python:
ci-rai-core-flask:
strategy:
# keep running remaining matrix jobs even if one fails
# to avoid having to rerun all jobs several times
Expand Down Expand Up @@ -47,12 +47,20 @@ jobs:
with:
python-version: ${{ matrix.pythonVersion }}

- name: Setup tools
- if: ${{ matrix.pythonVersion == '3.6' || matrix.pythonVersion == '3.7' }}
name: Setup tools for python lt 3.8
run: |
python -m pip install --upgrade pip
python -m pip install --upgrade "pip<=23.1.2"
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=6.13.0"
- if: ${{ matrix.pythonVersion == '3.8' || matrix.pythonVersion == '3.9' || matrix.pythonVersion == '3.10' }}
name: Setup tools for python gte 3.8
run: |
python -m pip install --upgrade pip
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=7.1.0"
- name: Pip compile
run: |
pip-compile requirements-dev.txt -o requirements-dev-comp.txt
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/CI-raiwidgets-pytest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ jobs:
run: |
python -m pip install --upgrade pip
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=6.13.0"
pip install --upgrade "pip-tools<=7.1.0"
- name: Install dependencies
shell: bash -l {0}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/CI-responsibleai-text-vision-pytest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ jobs:
run: |
python -m pip install --upgrade pip
pip install --upgrade setuptools
pip install --upgrade "pip-tools<=6.13.0"
pip install --upgrade "pip-tools<=7.1.0"
- name: Install dependencies
shell: bash -l {0}
Expand Down
6 changes: 4 additions & 2 deletions erroranalysis/erroranalysis/_internal/surrogate_error_tree.py
Original file line number Diff line number Diff line change
Expand Up @@ -471,12 +471,14 @@ def create_surrogate_model(analyzer,
surrogate = LGBMClassifier(n_estimators=1,
max_depth=max_depth,
num_leaves=num_leaves,
min_child_samples=min_child_samples)
min_child_samples=min_child_samples,
verbosity=-1)
else:
surrogate = LGBMRegressor(n_estimators=1,
max_depth=max_depth,
num_leaves=num_leaves,
min_child_samples=min_child_samples)
min_child_samples=min_child_samples,
verbosity=-1)
if cat_ind_reindexed:
surrogate.fit(dataset_sub_features, diff,
categorical_feature=cat_ind_reindexed)
Expand Down

0 comments on commit 0327bb2

Please sign in to comment.