Skip to content

Commit

Permalink
temp disable spectogram diffusion tests (huggingface#3278)
Browse files Browse the repository at this point in the history
The note-seq package throws an error on import because the default installed version of Ipython
is not compatible with python 3.8 which we run in the CI.
https://github.com/huggingface/diffusers/actions/runs/4830121056/jobs/8605954838#step:7:9
  • Loading branch information
williamberman authored and Jimmy committed Apr 26, 2024
1 parent c9abced commit e7188d5
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 0 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,6 @@
"Jinja2",
"k-diffusion>=0.0.12",
"librosa",
"note-seq",
"numpy",
"parameterized",
"protobuf>=3.20.3,<4",
Expand Down Expand Up @@ -191,7 +190,6 @@ def run(self):
"Jinja2",
"k-diffusion",
"librosa",
"note-seq",
"parameterized",
"pytest",
"pytest-timeout",
Expand Down
1 change: 0 additions & 1 deletion src/diffusers/dependency_versions_table.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@
"Jinja2": "Jinja2",
"k-diffusion": "k-diffusion>=0.0.12",
"librosa": "librosa",
"note-seq": "note-seq",
"numpy": "numpy",
"parameterized": "parameterized",
"protobuf": "protobuf>=3.20.3,<4",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,10 @@
MIDI_FILE = "./tests/fixtures/elise_format0.mid"


# The note-seq package throws an error on import because the default installed version of Ipython
# is not compatible with python 3.8 which we run in the CI.
# https://github.com/huggingface/diffusers/actions/runs/4830121056/jobs/8605954838#step:7:98
@unittest.skip("The note-seq package currently throws an error on import")
class SpectrogramDiffusionPipelineFastTests(PipelineTesterMixin, unittest.TestCase):
pipeline_class = SpectrogramDiffusionPipeline
required_optional_params = PipelineTesterMixin.required_optional_params - {
Expand Down

0 comments on commit e7188d5

Please sign in to comment.