Skip to content

Commit

Permalink
Split Jax training dependencies (especially Scenic and TF-IO) into a …
Browse files Browse the repository at this point in the history
…separate dependency group.

PiperOrigin-RevId: 614682564
  • Loading branch information
sdenton4 authored and copybara-github committed Mar 11, 2024
1 parent 3746672 commit df60818
Show file tree
Hide file tree
Showing 5 changed files with 275 additions and 228 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,6 @@ jobs:
run: |
sudo apt-get update
sudo apt-get install libsndfile1 ffmpeg
poetry install
poetry install --with jaxtrain
- name: Test with unittest
run: poetry run python -m unittest discover -s chirp/tests -p "*test.py"
35 changes: 35 additions & 0 deletions .github/workflows/ci_no_jaxtrain.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
name: CI_no_jaxtrain

on:
push:
branches: [ main ]
pull_request:
branches: [ main ]
# Allows to run this workflow manually from the Actions tab on GitHub.
workflow_dispatch:

jobs:
test-ubuntu:
name: "test on ${{ matrix.python-version }} on ${{ matrix.os }}"
runs-on: "${{ matrix.os }}"
strategy:
matrix:
python-version: ["3.10", "3.11"]
os: [ubuntu-latest]
steps:
- uses: actions/checkout@v3
- name: Set up Poetry
run: pipx install poetry
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v4
with:
python-version: ${{ matrix.python-version }}
cache: 'poetry'
- name: Install Chirp and its dependencies, except jaxtrain dependencies.
run: |
sudo apt-get update
sudo apt-get install libsndfile1 ffmpeg
poetry install --without jaxtrain
- name: Test with unittest
# TODO: Group together jaxtrain tests so they can be easily excluded.
run: poetry run python -m unittest discover -s chirp/tests -p "*inference_test.py"
1 change: 0 additions & 1 deletion chirp/inference/colab_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@

from absl import logging
from chirp import config_utils
from chirp.configs import config_globals
from chirp.inference import embed_lib
import numpy as np
import tensorflow as tf
Expand Down
Loading

0 comments on commit df60818

Please sign in to comment.