-
Notifications
You must be signed in to change notification settings - Fork 222
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
Error: "Message class can only inherit from Message" #7
Comments
Same here. But Comfy installed via 7zip file and extension via Manager. |
@Miosp @G-Grifflu I pushed the fix at 8b7ce6c. Can yall do a git pull? |
The error when opening Comfy is gone, but when trying to use the "MediaPipe face mesh preprocessor" this error is generated:
|
@Miosp @G-Grifflu I think the installation of mediapipe might have been wrong. Fixed at f99b09b |
That solved all the issues for me |
Still getting the error after these on my end. Running from a venv on linux miunt on its own ssd. Did a git clone --depth 1 --recurse-submodules --shallow-submodules https://github.com/Fannovel16/comfyui_controlnet_aux tried changing interp=img.LINEAR to img.Bilinear in src/custom_detectron2/data/transforms, same error. Delted pycache and rebooted - you guessed it. Added this under the load_custom_nodes() function based on another thread that said the error was due to protobuf loading something twice:
Bupkis. lemme know if there's anything I can help with to fix it. Not a python expert but I know a wee bit. |
@kronkinatorix Roop again 😒. Iirc, one of dependencies from roop installing a different version of protobuf, leading to conflicts with not just this extensions but also others. My extension doesn't make any change to protobuf. But I'll reopen this issue so that there won't be duplicated issues. As for now, that patch seems to be the only way fix it completely. |
@Fannovel16 Hi there! |
@Gourieff Is opencv-python 4.7.0.72 compatible with DWPose? A1111 side chooses 4.8.0 (Mikubill/sd-webui-controlnet#1897) |
Oh good, it works. Just did a quick test on my weak Windows machine but I think this is the case for other platforms. |
Same issue but with MeshGraphormer-DepthMapPreprocessor now
Yes i have Roop installed, |
me too ,and my opencv version 4.8.0.76 |
@mofos Same issue with MeshGraphormer, did you solve it? |
Nope but my best guess is a dependency conflict from my other custom nodes, my friends were able to get it running. i wish there was a way of identifying conflicts like this. if anyone knows how to do a proper debugging when my custom nodes have conflicts like this, please let me know. |
I've encountered the same issue (TypeError: A Message class can only inherit from Message). I've zeroed in on the origin of the message being deep in the venv and related to google.protobuf. The blow-by-blow can be viewed here. Roop - or some variant thereof - is a key module, so I'm reluctant to uninstall it. However, open to suggestions on how to resolve this. My system:
ETA: I've gone ahead and uninstalled roop. Installed Reactor and MeshGraphormer with no issues or errors. Seems clear that roop's dependency on some antiquated google.protobuf was the culprit. I'll leave this up in hope others might find their solution to a problem that has been confounding me for months. |
I know that this issue also existed in previous version of the extension and i never managed to get it working with methods mentioned earlier, so I'm gonna leave this there.
What I already tried:
My setup (something might mess things up and that's why it's not working):
The error:
The text was updated successfully, but these errors were encountered: