[tests] add tests for flux modular (t2i, i2i, kontext) by sayakpaul · Pull Request #12566 · huggingface/diffusers (original) (raw)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the PR
looks good, but I don't know why the changes in components_manager got reverted
I left a commenet there, I think we should add a warning during enable_model_oaffload(), and you can also keep the warning from the memory_reserve (or just make a note there in comment so we know that to fix in the fugure)

feel free to merge otherwise