From 966428453bd4afda75abd947b6b3e668a0cc42ed Mon Sep 17 00:00:00 2001 From: Spencer Krum Date: Mon, 16 Dec 2024 09:00:43 -0600 Subject: [PATCH 1/2] feat: test [all] install target in integration tests --- .github/workflows/integration.yaml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/.github/workflows/integration.yaml b/.github/workflows/integration.yaml index 82ca904c9..4328c33b3 100644 --- a/.github/workflows/integration.yaml +++ b/.github/workflows/integration.yaml @@ -52,6 +52,12 @@ jobs: - name: Install Built Package run: sdist-venv/bin/pip install --no-cache-dir "$(ls dist/validmind*.whl | head -n 1)[llm,huggingface]" + - name: 'Setup Virtual Environment for [all]' + run: python -m venv all-venv + + - name: 'Install Built Package for [all]' + run: all-venv/bin/pip install --no-cache-dir "$(ls dist/validmind*.whl | head -n 1)[all]" + - name: Install Additional Dependencies run: sdist-venv/bin/pip install nbformat papermill jupyter From 77461c73be0054b03cc3ece6663b4114f17f37a4 Mon Sep 17 00:00:00 2001 From: Spencer Krum Date: Mon, 16 Dec 2024 09:28:59 -0600 Subject: [PATCH 2/2] feat: add R dependencies --- .github/workflows/integration.yaml | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.github/workflows/integration.yaml b/.github/workflows/integration.yaml index 4328c33b3..da46c4020 100644 --- a/.github/workflows/integration.yaml +++ b/.github/workflows/integration.yaml @@ -32,7 +32,7 @@ jobs: python-version: '3.11' cache: 'poetry' - - name: Install Dependencies + - name: Install Python Dependencies run: | poetry config virtualenvs.in-project true poetry env use python3.11 @@ -40,6 +40,11 @@ jobs: poetry run pip install torch==2.0.1 --extra-index-url https://download.pytorch.org/whl/cpu poetry run pip install aequitas fairlearn vl-convert-python + - name: Install R Dependencies + run: | + sudo apt-get update + sudo apt install r-base r-base-dev + - name: Build the package run: poetry build @@ -55,6 +60,7 @@ jobs: - name: 'Setup Virtual Environment for [all]' run: python -m venv all-venv + # This proves that the [all] install target works - name: 'Install Built Package for [all]' run: all-venv/bin/pip install --no-cache-dir "$(ls dist/validmind*.whl | head -n 1)[all]"