-
Notifications
You must be signed in to change notification settings - Fork 50
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
PuLID Flux 0.9.1 Error on Sampler: "forward_orig() got an unexpected keyword argument 'attn_mask'" #122
Comments
Same here, I got this error after I updated ComfyUI and a bunch of custom nodes. File "Y:\Comfy\comfy\ldm\flux\model.py", line 204, in forward
out = self.forward_orig(img, img_ids, context, txt_ids, timestep, y, guidance, control, transformer_options, attn_mask=kwargs.get("attention_mask", None))
TypeError: forward_orig() got an unexpected keyword argument 'attn_mask' |
Same error. There are a bunch of people on the Comfyanonymous Git reporting the same issue, but they closed the ticket saying it's a custom code error. |
Same :( |
same |
+1 error |
But I did not even use that custom node... I dont' think it'd solve the error I'm getting. |
同样的问题,前几天好好着呢,今天就同样的出现自定义采样器(高级) |
the new version of comfyui 0.3.8 still doesn't solve the problem |
I just tried and it seems to be working for me. I would need a more detailed error report including the full back trace, system information and library versions |
Making the changes described in this person's code fix made it work correctly for me. |
Well ... For the moment, as workaround, I removed attn_mask=kwargs.get("attention_mask", None) from
If you still have errors with this node you can try removing transformer_options as well. For this one, you can find a fix here. Please be advised, for the ComfyUI-PuLID-Flux-Enhanced node we are still waiting for sipie800 to merge PR41 which should fix "Fix tensors in different device error." Maybe it's not the best solution, but I hope it helps... |
@cubiq , given that sipie800 may no longer have the time to manage ComfyUI-PuLID-Flux-Enhanced node, perhaps you could consider forking it (or merging into PuLID_ComfyUI, as a very useful extension for FLUX) and addressing the recent issues. |
Once again, this is the wrong repository. The flux PuLID node is sipie800/ComfyUI-PuLID-Flux-Enhanced. |
Download the `ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-PuLID-Flux-Enhanced' |
Can confirm that this works. |
生效了 |
It works perfectly as you said! Thank you! |
完美 |
didn't work for me, not even replacing the files in the ldm folder with previous version worked and i am on 0.39 |
上述pulidflux,py确实解决了问题,但导致了我重新出现了Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu |
无法上传zip,你可以自行合并2个pulidflux,py(简单的方法是哪个单词多,就用哪个部分) |
replaced pulidflux.py and got another error ComfyUI Error ReportError Details
Stack Trace
System Information
Devices
Logs
Attached WorkflowPlease make sure that workflow does not contain any sensitive information such as API keys or passwords.
Additional Context(Please add any additional context or steps to reproduce the error here) |
Please try to check the torch torchaudio and torchversion and xformer version, I reinstall them and it works well now. |
老哥,可否上传一下百度网盘或夸克盘,不是程序员,不太清楚这两个版本的文件你们各自改了啥地方,不太懂这方面。 |
updating nodes solved my issue |
I noticed in
ComfyUI_windows_portable\ComfyUI\comfy\ldm\flux\model.py, there's a line toward the bottom:
out = self.forward_orig(img, img_ids, context, txt_ids, timestep, y, guidance, control, transformer_options, attn_mask=kwargs.get("attention_mask", None))
So I wonder why it was giving the issue...
The text was updated successfully, but these errors were encountered: