Repository navigation
Closed
Labels
bugSomething isn't workingSomething isn't working
Description
Describe the bug
HiDream pipeline is currently broken on diffusers main branch due to PR #11296 that was merged few days ago.
it introduces a bad deprecation check which simply fails on main branch.
Reproduction
N/A
Logs
│ /home/vlado/dev/sdnext/venv/lib/python3.12/site-packages/diffusers/models/transformers/transformer_hidream_image.py:794 in forward │
│ │
│ 793 │ │ │ deprecation_message = "The `encoder_hidden_states` argument is deprecated. Please use `encoder_hidden_states_t5` and `encoder_hidden_states_llama3` instead." │
│ ❱ 794 │ │ │ deprecate("encoder_hidden_states", "0.34.0", deprecation_message) │
│ 795 │ │ │ encoder_hidden_states_t5 = encoder_hidden_states[0] │
│ │
│ /home/vlado/dev/sdnext/venv/lib/python3.12/site-packages/diffusers/utils/deprecation_utils.py:18 in deprecate │
│ │
│ 17 │ │ if version.parse(version.parse(__version__).base_version) >= version.parse(version_name): │
│ ❱ 18 │ │ │ raise ValueError( │
│ 19 │ │ │ │ f"The deprecation tuple {(attribute, version_name, message)} should be removed since diffusers'" │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
ValueError: The deprecation tuple ('encoder_hidden_states', '0.34.0', 'The `encoder_hidden_states` argument is deprecated. Please use `encoder_hidden_states_t5` and `encoder_hidden_states_llama3` instead.') should be removed since diffusers' version 0.34.0.dev0 is >= 0.34.0System Info
diffusers==main
Who can help?
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working
Activity
linoytsaban commentedon Apr 22, 2025
Hi @vladmandic, which
transformersversion are you using?HiDream inference from main works fine for me
vladmandic commentedon Apr 22, 2025
ok, it breaks custom hidream pipelines that were copied/modified and not updated to absolutely latest codebase.
i was using hidream cfgzero pipeline and it fails because it uses old-style
encode_hidden_statesformat:so yes, this is not affecting built-in pipeline. but deprecation is veeeery aggressive, not giving any chance downstream community to catch-up.
[-]HiDream is broken on main[/-][+]HiDream pipeline update breaks downstream[/+]yiyixuxu commentedon Apr 22, 2025
hi @vladmandic let me move it to 0.35.0 (it was intended for after next release anyway)
vladmandic commentedon Apr 22, 2025
lgtm, thanks!