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

Default model settings #5850

Merged
merged 12 commits into from
Mar 4, 2024
Merged

Default model settings #5850

merged 12 commits into from
Mar 4, 2024

Conversation

maryhipp
Copy link
Collaborator

@maryhipp maryhipp commented Mar 1, 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

Adds a subset of default settings to model management that a user can easily override their settings with when generating.

QA Instructions, Screenshots, Recordings

Screenshot 2024-02-29 at 4 26 10 PM
Screenshot 2024-02-29 at 4 26 25 PM

@github-actions github-actions bot added api python PRs that change python files backend PRs that change backend files services PRs that change app services frontend PRs that change frontend files labels Mar 1, 2024
@maryhipp maryhipp force-pushed the maryhipp/default-model-settings-main branch from 18cdb05 to 0e92bca Compare March 1, 2024 23:38
@hipsterusername hipsterusername force-pushed the maryhipp/default-model-settings-main branch from 0e92bca to 7f73ca9 Compare March 2, 2024 13:37
@maryhipp maryhipp enabled auto-merge (rebase) March 4, 2024 13:45
auto-merge was automatically disabled March 4, 2024 14:38

Rebase failed

@maryhipp maryhipp merged commit 8b34f52 into main Mar 4, 2024
14 checks passed
@maryhipp maryhipp deleted the maryhipp/default-model-settings-main branch March 4, 2024 14:39
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably use the same metadata handling logic as the rest of the app. No need to re-implement all that here.

RyanJDick pushed a commit that referenced this pull request Mar 8, 2024
* UI in MM to create trigger phrases

* add scheduler and vaePrecision to config

* UI for configuring default settings for models'

* hook MM default model settings up to API

* add button to set default settings in parameters

* pull out trigger phrases

* back-end for default settings

* lint

* remove log;
gi

* ruff

* ruff format

---------

Co-authored-by: Mary Hipp <maryhipp@Marys-MacBook-Air.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api backend PRs that change backend files frontend PRs that change frontend files python PRs that change python files services PRs that change app services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants