text-generation-webui icon indicating copy to clipboard operation
text-generation-webui copied to clipboard

Dual fp16 / GGUF models not downloaded correctly

Open PyroGenesis opened this issue 2 years ago • 0 comments

Describe the bug

When I try to download a model whose repo as both fp16 weights and GGUF files, it incorrectly downloads the model files to the models directory instead of a subfolder.

Is there an existing issue for this?

  • [X] I have searched the existing issues

Reproduction

Try to download defog/sqlcoder-7b-2 leaving specific file as empty

Details

I looked into the code a bit, and maybe this is a '' vs None issue.

download_model_wrapper() passes specific_file='' to downloader.get_download_links_from_huggingface() https://github.com/oobabooga/text-generation-webui/blob/0f134bf744acef2715edd7d39e76f865d8d83a19/modules/ui_model_menu.py#L245-L251

but get_download_links_from_huggingface() checks if specific_file is not None https://github.com/oobabooga/text-generation-webui/blob/0f134bf744acef2715edd7d39e76f865d8d83a19/download-model.py#L155-L157

BTW this wasn't an issue in an older version of text-generation-webui

System Info

OS: Windows Server 2022

PyroGenesis avatar Feb 12 '24 21:02 PyroGenesis