Skip to content

Altered ExecutionList by further grouping [less model reload] #7978

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

XmYx
Copy link

@XmYx XmYx commented May 7, 2025

Allow me to share, and in case it looks interesting, please test and excuse for linting issues, i shall be able to work on those, but the idea is that we could group the nodes by the required models, this, for many cases could potentially yield a lower execution time in my understanding, by mitigating the amount of model reloads needed, but allow me to make a mistake, and please point me in the right direction with the aim for me to be able to help, and improve it if it looks like a path to follow.

173.2s -> 142.1s [18%?]

Thank you

discord: mix

XmYx added 2 commits May 7, 2025 20:10
Improved execution to mitigate model reloads in large graphs
ask mix on Discord for further information please
@XmYx XmYx requested a review from comfyanonymous as a code owner May 7, 2025 18:14
@XmYx XmYx changed the title Improved executionlist by grouping staged nodes Improved executionlist by further grouping [less model reload] May 7, 2025
@XmYx XmYx changed the title Improved executionlist by further grouping [less model reload] Altered ExecutionList by further grouping [less model reload] May 7, 2025
@XmYx
Copy link
Author

XmYx commented May 9, 2025

bumping for vis

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