Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

convert.py/patch refactor #216

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

convert.py/patch refactor #216

wants to merge 3 commits into from

Conversation

city96
Copy link
Owner

@city96 city96 commented Feb 20, 2025

This currently adds:

  • The updated patch that has ltxv and hunyuan key names (replaces the special SD3 patch to make it less confusing)
  • The extra script required for the janky 4 step process required by hunyuan + the readme for it
  • Some commits from a month ago that reduce the number of tensors kept in FP32 + add sanity checks

Not 100% confident in merging yet, will add other model support here & do a full sweep of the other models before this can be merged.

Possible TODOs/ideas:

  • Use comfy native model detection logic for convert script
  • clean up fix_5d_tensors.py

@city96
Copy link
Owner Author

city96 commented Feb 20, 2025

Tagging @kijai and @pollockjj who requested the updated code - this is the same as what I shared earlier but you guys can follow the PR for updates.

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