-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Add defaults for controlnet/textual inversion/realesrgan #3773
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
Conversation
- remove restoration services - remove the restore faces nodes - update tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Lincoln, I've added and tested a RealESRGAN upscaling node, works great.
We can fully remove the restoration services now, they are not needed. I've removed all references to them in the nodes app layer and updated the tests.
It's just the backend that has the actual restoration code - I'll leave that to you to remove.
@maryhipp @brandonrising @ebr
It also introduces an upscaling node, which is not yet exposed in the Linear UI. I'll add that later. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I've removed all the backend restoration code and will merge. |
I've added a couple small fixes:
|
This PR adds several default models to the ones selected at install time. It also removes the GFPGAN and text2clip models, which should shave a little time off the install process.
ESRGAN:
ControlNet
Embedding (textual inversion)