Skip to content

Commit

Permalink
Fix lightgbm slow test (mlflow#13697)
Browse files Browse the repository at this point in the history
Signed-off-by: Weichen Xu <[email protected]>
  • Loading branch information
WeichenXu123 authored Nov 7, 2024
1 parent c363c67 commit bee39c9
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/slow-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,8 @@ jobs:
source ./dev/install-common-deps.sh --ml
# transformers requires `tf-keras` when keras 3.0 is installed.
# See https://github.com/huggingface/transformers/issues/27377 for more details.
python -m pip install langchain_experimental tf-keras
# Installing pyarrow < 18 to prevent lightgbm test pip installation resolution conflicts.
python -m pip install langchain_experimental tf-keras "pyarrow<18"
- uses: ./.github/actions/show-versions
- uses: ./.github/actions/pipdeptree
- name: Run tests
Expand Down
3 changes: 3 additions & 0 deletions tests/pyfunc/docker/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,9 @@ def save_model_with_latest_mlflow_version(flavor, extra_pip_requirements=None, *
else:
extra_pip_requirements = extra_pip_requirements or []
extra_pip_requirements.append(f"mlflow=={latest_mlflow_version}")
if flavor == "lightgbm":
# Adding pyarrow < 18 to prevent pip installation resolution conflicts.
extra_pip_requirements.append("pyarrow<18")
kwargs["extra_pip_requirements"] = extra_pip_requirements
flavor_module = getattr(mlflow, flavor)
flavor_module.save_model(**kwargs)

0 comments on commit bee39c9

Please sign in to comment.