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 'interrogate' and 'all' choices to --use-cpu, make --use-cpu case insensitive #2597

Merged

Conversation

brkirch
Copy link
Collaborator

@brkirch brkirch commented Oct 14, 2022

  • Add 'interrogate' and 'all' choices to --use-cpu
  • Change type for --use-cpu argument to str.lower, so that choices are case insensitive

Interrogate doesn't currently work correctly on MPS so adding it as a choice to the --use-cpu option provides a way to use that feature without also forcing Stable Diffusion onto CPU.

Adding the choice of 'all' to --use-cpu is to make the option more convenient for anyone who wants to run as much as possible on CPU.

* Add 'interrogate' and 'all' choices to --use-cpu
* Change type for --use-cpu argument to str.lower, so that choices are case insensitive
@brkirch brkirch changed the title Add 'interrogate' and 'all' choices to --use-cpu Add 'interrogate' and 'all' choices to --use-cpu, make --use-cpu case insensitive Oct 14, 2022
@AUTOMATIC1111
Copy link
Owner

AUTOMATIC1111 commented Oct 14, 2022

type=str.lower

clever

@AUTOMATIC1111 AUTOMATIC1111 merged commit fdef825 into AUTOMATIC1111:master Oct 14, 2022
@brkirch brkirch deleted the add-interrogate-and-all-to-use-cpu branch October 14, 2022 17:39
Atry pushed a commit to Atry/stable-diffusion-webui that referenced this pull request Jul 11, 2024
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.

2 participants