GreenLandisaLie / AuraSR-ComfyUI

ComfyUI implementation of AuraSR
Creative Commons Attribution Share Alike 4.0 International
144 stars 5 forks source link

Can't load model name #6

Closed ShineHe2023 closed 2 months ago

ShineHe2023 commented 3 months ago

Snipaste_2024-07-03_09-21-11 Instructions: Create a folder named 'Aura-SR' inside '\models'. Download the .safetensors AND config.json files from HuggingFace and place them in '\models\Aura-SR'

GreenLandisaLie commented 3 months ago

Reload the page. If that still doesn't work then reload ComfyUI entirely.

EDIT: Just tried it out. So long as you don't have any node that's not changing ComfyUI's internal 'models' path - this can only happen if you add the AuraSR node before adding the models to the Aura-SR folder. Once the models are in there, you only need to reload the browser's page and click on the white arrow '>' to select the model. At this point, the model will be selected by default on newly added AuraSR nodes.

ShineHe2023 commented 3 months ago

Reload the page. If that still doesn't work then reload ComfyUI entirely.

EDIT: Just tried it out. So long as you don't have any node that's not changing ComfyUI's internal 'models' path - this can only happen if you add the AuraSR node before adding the models to the Aura-SR folder. Once the models are in there, you only need to reload the browser's page and click on the white arrow '>' to select the model. At this point, the model will be selected by default on newly added AuraSR nodes.

I restart ComfyUI many times,and refreshed browser many times, but it doesn't work

lonmusk commented 3 months ago

I have the same trouble

mvstrawn commented 3 months ago

Same issue. I restarted everything, uninstalled/reinstalled the plugin (I had not followed the installation order and installed the node prior to creating the /models/Aura-SR/ folder), restarted again.

But I fixed it!

In nodes.py,

folder_paths.folder_names_and_paths["aura-sr"] = ([os.path.join(folder_paths.models_dir, "aura-sr")], folder_paths.supported_pt_extensions)

is looking for 'aura-sr', not 'Aura-SR'. I am on Arch Linux, where pathing is case sensitive. Changing the folder name to lower case fixed it.

GreenLandisaLie commented 3 months ago

@mvstrawn Thank you for the insights. If it works on some OSs and not on others then this is a Python issue in my opinion. I'll fix it right away. Hopefully it works for OP too even though he seems to be on Windows.

GreenLandisaLie commented 3 months ago

@ShineHe2023, @lonmusk I've updated with a fix for case sensitive pathing. Please install the latest version via Git URL (because the Manager takes some time to update after a repo change) following a ComfyUI restart then refresh the browser's page and delete any AuraSR node that shows up by default from your cache - if they show up. Add a new AuraSR node and check if its working properly now.

IF its still not working then I need to check your paths for anything that may be causing an issue and for that I need you to do the following:

GreenLandisaLie commented 2 months ago

Closing this under the assumption it has already been fixed in the latest update.