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

add --ignore_missing_core_models CLI flag to bypass checking for missing core models #4081

Conversation

damian0815
Copy link
Contributor

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

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

As per title - if InvokeAI won't start because you don't have some core conversion models installed, and you know you will not need them, you can pass --ignore_missing_core_models on startup and the check will be bypassed.

Copy link
Collaborator

@lstein lstein left a comment

Choose a reason for hiding this comment

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

Looks good. Just need to tell the user the consequence of not installing the core models as noted in the code comment below.

invokeai/backend/install/check_root.py Outdated Show resolved Hide resolved
@psychedelicious
Copy link
Collaborator

@lstein Need you to re-approve now that your requested changes have been addressed.

Copy link
Collaborator

@lstein lstein left a comment

Choose a reason for hiding this comment

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

LGTM

@psychedelicious psychedelicious enabled auto-merge (rebase) August 5, 2023 01:58
@psychedelicious psychedelicious merged commit eb6c317 into invoke-ai:main Aug 5, 2023
8 checks passed
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.

3 participants