search

mikonvergence / ControlNetInpaint

Inpaint images with ControlNet
MIT License
344 stars 29 forks source link

Fix support for diffusers 0.15.0 #7

Closed remorses closed 1 year ago

remorses commented 1 year ago

To support latest diffusers you also need to wait for this PR

mikonvergence commented 1 year ago

Yeah, I just realised the positional arg do_classifier_free_guidance is also causing problems with the latest release.

Let's see if that can be fixed in the PR you mentioned on the diffusers side. If it doesn't get resolved in the next week or so, we can adjust the code in this repo.

For now, I will make it explicit that diffusers==0.14.0 is currently supported.

mikonvergence commented 1 year ago

@remorses It seems that the issue does not occur with 0.16.0? Is it okay to close this down?

remorses commented 1 year ago

I would merge this PR because direct use of model parameters is discouraged, it’s better to use the config object

feel free to close the PR if you don’t want to

mikonvergence commented 1 year ago

Right, there is that too. I agree, will merge it then.