-
-
Notifications
You must be signed in to change notification settings - Fork 367
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
new v1.29 is still looking for MAT_Places512_G_fp16 inside the old Comfy models/inpaint folder #1469
Comments
After posting the above I found that the same thing applies to the inpaint fooocus models for SDXL and some other model files. |
Relevant information:
Not really sure about the mixup between managed/custom install, only way that's possible is if the |
Ok, here goes. The update log section: [server-path]/models/inpaint] C:\AI\ai_diffusion.serverI\models\inpaint>dir Directory of C:\AI\ai_diffusion.serverI\models\inpaint 06/12/2024 20:37 .19/10/2024 12:14 .. 13/02/2024 17:29 998 config.json 13/02/2024 18:32 1.445.157.124 diffusion_pytorch_inpaint.safetensors 13/02/2024 15:53 52.602 fooocus_inpaint_head.pth 29/06/2024 22:48 102.218.542 fooocus_lama.safetensors 13/02/2024 15:55 1.323.362.033 inpaint_v26.fooocus.patch 29/06/2024 21:08 2.502.139.104 Kataragi_inpaintXL-fp16.safetensors 13/02/2024 15:50 125.280.278 mat_places512_g_fp16.safetensors 13/07/2024 20:13 32.768 model_metadata.ldb 8 File(s) 5.498.243.449 bytes 2 Dir(s) 498.850.418.688 bytes free [server-path]/ComfyUI/models/inpaint] C:\AI\ai_diffusion.serverI\ComfyUI\models\inpaint>dir Directory of C:\AI\ai_diffusion.serverI\ComfyUI\models\inpaint 06/12/2024 20:37 .07/12/2024 00:33 .. 13/02/2024 17:29 998 config.json 13/02/2024 18:32 1.445.157.124 diffusion_pytorch_inpaint.safetensors 29/06/2024 22:48 52.602 fooocus_inpaint_head.pth 29/06/2024 22:48 102.218.542 fooocus_lama.safetensors 29/06/2024 22:45 1.323.362.033 inpaint_v26.fooocus.patch 29/06/2024 21:08 2.502.139.104 Kataragi_inpaintXL-fp16.safetensors 13/02/2024 15:50 125.280.278 mat_places512_g_fp16.safetensors 13/07/2024 20:13 32.768 model_metadata.ldb 8 File(s) 5.498.243.449 bytes 2 Dir(s) 498.858.962.944 bytes free [server-path]/ComfyUI/extra_model_paths.yaml (renamed to extra_model_paths.yaml.txt just to be able to drop it here] |
Also on this subject: I could never get the plugin use the clip_l model in my config, it is always listed as not found in the client.log, no matter where I place it (it is now in about 6 locations, including those under Stability Matrix, still the same): C:\AI\ai_diffusion.serverI\models\clip_vision>dir /s Directory of C:\AI\ai_diffusion.serverI\models\clip_vision 06/12/2024 21:36 .19/10/2024 12:14 .. 06/12/2024 21:30 246.144.152 clip_l.safetensors 17/10/2024 18:18 3.689.911.098 clip_vision_g.safetensors 13/02/2024 15:40 0 put_clip_vision_models_here 18/10/2024 22:18 SD1.5 3 File(s) 3.936.055.250 bytes Directory of C:\AI\ai_diffusion.serverI\models\clip_vision\SD1.5 18/10/2024 22:18 .06/12/2024 21:36 .. 13/02/2024 15:50 2.528.373.448 model.safetensors 1 File(s) 2.528.373.448 bytes
The last occurrence: 2024-12-07 00:30:37,687 INFO Optional Text Encoder model clip_l for All not found (search path: clip_l) What's the deal with it? |
And while we are on this... a new nuisance after installing v1.29: every time I do Connect to my custom Comfy server from the plugin's Configure dialogue, I notice this message in the server console (after a normal, error-free startup sequence): To see the GUI go to: http://0.0.0.0:8190 I don't recall seeing this error before installing v1.29. This doesn't seem to affect the plugin's work (everything seems normal from within Krita AI), but I wonder why I am getting this, not having installed Flux at all or anything else gguf-related. |
This looks all good as far as I can tell. Update worked, it migrated models to the new path ( The old model folder (
It goes to the
I'll fix that (shouldn't print an error), but yes it doesn't have any impact since it just says it didn't find any GGUF models (which you don't have). So I can't find anything wrong here? Was your error with custom StabilityMatrix comfy? I can't really help with that. If you fixed it by moving the models to the old folder, I can only assume that you at some point set it up that way! And it no longer worked because the updated moved the folder. That's to be expected. |
The missing MAT_Places512_G_fp16 failure - as I found out now - didn’t have anything to do with Stability Matrix in the picture, but with the fact that the external Comfy was/is using junctions for folders in the plugin-managed one's folder tree (to avoid duplication), which were nullified during the v1.29 installation. But I think this is less relevant than it might sound, since it was just one of many examples of this kind of situation that I hear almost daily from folks on Discord (whom I am always glad to help, but my knowledge has its limits of course). I’d really like to see more help from the log/diagnostic system of the plugin for both types of configuration, especially in show-stopping situations like this one. For starters. it would be much more helpful if during the startup sequence the plugin added to the error message strings like “(partial) match: MAT_Places512_G_fp16 …” and “INFO Optional Text Encoder model clip_l for All not found (search path: clip_l)” in the log file the folder name(s) where the plugin was looking for these components. Would this be feasible? That would save lots of guessing work for many users installing or updating the plugin, or otherwise modifying the setup. It goes to the clip folder, not clip_vision This worked, thanks! Could you shed a bit of light on which base models use this file and for what? |
Would be nice, but not so easy to have. The plugin doesn't know the folders. Would have to extend Comfy to query them. Clip-L is used by pretty much all models, but in most cases it's part of the checkpoint |
After installing the new v1.29, I spent half an hour resolving the red-colored error message showing up during plugin's startup about missing mat_places512_g_fp16 file. I put a copy of the file everywhere I could think of in both managed and external trees (about 6 copies in total), the plugin still wouldn't find it. It's only after I placed it inside the old Comfy/models tree of the managed server (in the inpaint subfolder) that it could start without an error. It's also strange because I was trying to start the plugin using an external Comfy server, not the managed one.
The text was updated successfully, but these errors were encountered: