Diffuser test#2141
Conversation
|
bitsandbytes==0.44 fails with |
|
diffuser fails with bitsandbytes==0.45.0 diffuser works with bitsandbytes==0.45.1, but |
|
@kiya00 the error is fixed in the bitsandbytes main branch, but we're waiting for the 0.47 release to bump. (@lianakoleva isolated this in #2238 ) |
|
Hi @t-vi , thanks, I think it's fine to wait for 0.47 to merge this PR, I'll ask @IvanYashchuk on Monday |
What's the stack trace for this error? I don't see |
|
|
Hi @t-vi @IvanYashchuk ,it's ready to merge |
t-vi
left a comment
There was a problem hiding this comment.
Thank you @kiya00
As much as we love broad coverage testing, we need to figure out how to reduce the test time.
148.69s call thunder/tests/test_networks.py::test_hf_diffusers[ionet-official/bc8-alpha]
148.37s call thunder/tests/test_networks.py::test_hf_diffusers[runwayml/stable-diffusion-v1-5]
148.29s call thunder/tests/test_networks.py::test_hf_diffusers[stabilityai/stable-diffusion-xl-refiner-1.0]
147.92s call thunder/tests/test_networks.py::test_hf_diffusers[runwayml/stable-diffusion-inpainting]
146.58s call thunder/tests/test_networks.py::test_hf_diffusers[CompVis/stable-diffusion-v1-4]
145.77s call thunder/tests/test_networks.py::test_hf_diffusers[stabilityai/stable-diffusion-xl-base-1.0]
144.68s call thunder/tests/test_networks.py::test_hf_diffusers[stabilityai/sd-turbo]
144.28s call thunder/tests/test_networks.py::test_hf_diffusers[diffusers/stable-diffusion-xl-1.0-inpainting-0.1]
|
Hi @t-vi , Can I move it into |
That is expected, as you try to execute the CUDA test on a plain CPU machine/runner without any CUDA support, you would need to add it to Azure or LitCI jobs |
for more information, see https://pre-commit.ci
|
Hi @t-vi , @Borda , the diffuser test passed (the |
|
now failing with the missing argument in the used models: |
Hi @Borda @t-vi , at first glance, each model appears to be failing for a different reason. Fixing them may take some time. Since this PR is only for the diffuser test, can we review and merge this one first, and then open a separate PR to fix |
Before submitting
What does this PR do?
Fixes #2075 .
Adds coverage tests for HF diffusers
Needs #2122: The bitsandbytes version needs to be updated to support HF Diffusers, as was done in PR #2122 .
cc @Borda