Skip to content

Modernize Compel for Transformers 5 and add diffusers smoke coverage#1

Merged
lstein merged 1 commit into
mainfrom
apply-pr129-transformers5
May 29, 2026
Merged

Modernize Compel for Transformers 5 and add diffusers smoke coverage#1
lstein merged 1 commit into
mainfrom
apply-pr129-transformers5

Conversation

@lstein
Copy link
Copy Markdown

@lstein lstein commented May 29, 2026

Applies upstream PR damian0815/compel#129 ("Modernize Compel for Transformers 5 and add diffusers smoke coverage") to this fork.

Changes

  • Raise support floor to Python >=3.10; align CI matrix (drop 3.9)
  • Update transformers guidance to >=5,<6; pin pyparsing~=3.0
  • Migrate deprecated pyparsing API (delimited_listDelimitedList)
  • Remove stray networkx import in convenience_wrappers.py
  • Add public-API / CLIP / SD / SDXL / T5 smoke tests and diffusers prompt_embeds / pooled_prompt_embeds coverage (incl. opt-in local SDXL single-file checkpoint tests gated behind COMPEL_RUN_LOCAL_CHECKPOINT_TESTS=1)
  • Add provider tests for pooled output handling and BOS-less T5 behavior
  • README cleanup and example typo fixes (negative_prompt_embednegative_prompt_embeds)

Validation

  • Created a fresh venv (Python 3.12.3), editable install pulled transformers 5.9.0, diffusers 0.38.0, pyparsing 3.3.2
  • PYTHONPATH=src:test python -m unittest discover -s test61 passed, 1 skipped (the opt-in local checkpoint test)

⚠️ Note for reviewers

This raises the dependency floor to transformers >=5,<6. Please confirm compatibility with InvokeAI's own transformers pins before merging into main.

🤖 Generated with Claude Code

Apply upstream PR damian0815#129:
- raise support floor to Python >=3.10 and align CI matrix
- update transformers guidance to >=5,<6; pin pyparsing~=3.0
- migrate deprecated pyparsing API (delimited_list -> DelimitedList)
- remove stray networkx import in convenience_wrappers
- add public-API/CLIP/SD/SDXL/T5 smoke tests and diffusers
  prompt_embeds/pooled_prompt_embeds coverage (incl. opt-in local
  SDXL single-file checkpoint tests)
- add provider tests for pooled output and BOS-less T5 behavior
- README cleanup and example typo fixes

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@lstein lstein self-assigned this May 29, 2026
@lstein lstein merged commit 8f404b4 into main May 29, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant