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

Next: Fix Compel node TI loading #5813

Closed
wants to merge 1 commit into from

Conversation

brandonrising
Copy link
Collaborator

@brandonrising brandonrising commented Feb 27, 2024

…never be the case.

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Community Node Submission

Have you discussed this change with the InvokeAI team?

  • Yes
  • No, because:

Have you updated all relevant documentation?

  • Yes
  • No

Description

Currently the compel node is trying to load TIs using the name in the prompt as the key. This should never be the case. This PR updates the code to load via attributes.

@github-actions github-actions bot added python PRs that change python files invocations PRs that change invocations labels Feb 27, 2024
@brandonrising brandonrising force-pushed the next-fix-ti-loading-in-compel branch from b6f0048 to 4d6047e Compare February 27, 2024 15:05
@brandonrising
Copy link
Collaborator Author

oh wait nevermind, apparently the UI is injecting the key into the prompt now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invocations PRs that change invocations python PRs that change python files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant