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

Incompatible with v1.6.0 of webui due to prompt parser change #60

Closed
catboxanon opened this issue Aug 25, 2023 · 5 comments · Fixed by #61
Closed

Incompatible with v1.6.0 of webui due to prompt parser change #60

catboxanon opened this issue Aug 25, 2023 · 5 comments · Fixed by #61

Comments

@catboxanon
Copy link

Posting this on behalf of someone else -- essentially though, a recent PR for the prompt parser that was merged right before 1.6.0-RC was released, broke this extension.

Traceback (most recent call last):
      File "F:\stable-diffusion-webui\modules\call_queue.py", line 57, in f
        res = list(func(*args, **kwargs))
      File "F:\stable-diffusion-webui\modules\call_queue.py", line 36, in f
        res = func(*args, **kwargs)
      File "F:\stable-diffusion-webui\modules\txt2img.py", line 55, in txt2img
        processed = processing.process_images(p)
      File "F:\stable-diffusion-webui\modules\processing.py", line 732, in process_images
        res = process_images_inner(p)
      File "F:\stable-diffusion-webui\extensions\sd-webui-controlnet\scripts\batch_hijack.py", line 42, in processing_process_images_hijack
        return getattr(processing, '__controlnet_original_process_images_inner')(p, *args, **kwargs)
      File "F:\stable-diffusion-webui\modules\processing.py", line 856, in process_images_inner
        p.setup_conds()
      File "F:\stable-diffusion-webui\modules\processing.py", line 1315, in setup_conds
        super().setup_conds()
      File "F:\stable-diffusion-webui\modules\processing.py", line 469, in setup_conds
        self.uc = self.get_conds_with_caching(prompt_parser.get_learned_conditioning, negative_prompts, total_steps, [self.cached_uc], self.extra_network_data)
      File "F:\stable-diffusion-webui\modules\processing.py", line 455, in get_conds_with_caching
        cache[1] = function(shared.sd_model, required_prompts, steps, hires_steps, shared.opts.use_old_scheduling)
      File "F:\stable-diffusion-webui\extensions\prompt-fusion-extension\lib_prompt_fusion\hijacker.py", line 15, in wrapper
        return function(*args, **kwargs, original_function=self.__original_functions[attribute])
    TypeError: _hijacked_get_learned_conditioning() got multiple values for argument 'original_function'

Related PR: AUTOMATIC1111/stable-diffusion-webui#12457

@catboxanon catboxanon changed the title Incompatible with v1.6.0 of webui Incompatible with v1.6.0 of webui due to prompt parser change Aug 25, 2023
@ljleb
Copy link
Owner

ljleb commented Aug 25, 2023

It's too late at the moment, will fix tomorrow.

@pmcculler
Copy link

I'm a passionate user of prompt-fusion and I don't think I can work with A1111's 1.6 release candidate until it's supported. I'm a developer, is there anything I can do to assist?

@ljleb
Copy link
Owner

ljleb commented Aug 30, 2023

Hey, thanks for landing a hand. I spent a bit of time to fix this + sdxl this night actually. I'm almost done, I still have to move some of the code around and make a path for some of the values. I'll try to look into it a little bit more tomorrow ™️

@pmcculler
Copy link

Well, marvelous! I'll keep an eye out then. :)

@ljleb
Copy link
Owner

ljleb commented Aug 31, 2023

This should be fixed now, please open a new issue if there is any problem with the fix.

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 a pull request may close this issue.

3 participants