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

Fix resolution | None | Add chedule_setting for init_image #14

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

NeuralAIM
Copy link

Fix for resolution <64px
image

@NeuralAIM NeuralAIM changed the title Fix resolution change Fix resolution | None | Add chedule_setting for init_image Jun 20, 2022
@NeuralAIM
Copy link
Author

NeuralAIM commented Jun 20, 2022

It will be easier if the user immediately finds out what kind of video card he has:

image
p.s. And also simplified the output.


It was inconvenient to change the settings every time you add or delete init_image. I think it needs to be added to the config settings:

#Maximum step is 25, larger - create artifacts
custom_schedule_setting = [
 [200,1000,8],
 "gfpgan:1.5","scale:.9","noise:.55",
 [1,200,5],
]

#For init image
custom_schedule_setting_init = [
 [100,500,8], #It consists of 50% init_image and 50% model generation
 [1,100,5],
]

Added a fix for colab when None was considered as string

def NoneToNone(par): #fix for None(str)
    if(par == '' or par.lower() == 'none'):
        par = None
    return par

I tried to translate the text into English correctly, but maybe there are mistakes somewhere. I'm not a programmer, but I decided to try to help a little, don't judge strictly :)

@apolinario
Copy link
Contributor

Thanks so much for those improvements @NeuralAIM! One thing I didn't get for the init_image is why did you need to add those xtra steps? I currently modify the first step to skip the steps directly in the schedule, could you go over a bit why you did it that way?

@NeuralAIM
Copy link
Author

Thanks so much for those improvements @NeuralAIM! One thing I didn't get for the init_image is why did you need to add those xtra steps? I currently modify the first step to skip the steps directly in the schedule, could you go over a bit why you did it that way?

starting_timestep - It didn't work for me. I fixed it :)

@Dango233 Dango233 added the enhancement New feature or request label Jul 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants