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

updates for defaultModel #5866

Merged
merged 2 commits into from
Mar 5, 2024
Merged

updates for defaultModel #5866

merged 2 commits into from
Mar 5, 2024

Conversation

maryhipp
Copy link
Collaborator

@maryhipp maryhipp commented Mar 5, 2024

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

  • move defaultModel logic to modelsLoaded - if user does not have a model in their state already (typically after a web UI refresh or first load), it will use the passed in defaultModel key if it is in their list of main models. it will also fire the optimal width/height logic for that model type.
  • update to work for key instead of name/base/type string

@github-actions github-actions bot added the frontend PRs that change frontend files label Mar 5, 2024
@maryhipp maryhipp merged commit e30cb4b into main Mar 5, 2024
14 checks passed
@maryhipp maryhipp deleted the maryhipp/default-model-updates branch March 5, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend PRs that change frontend files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants