Skip to content
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

[enhancement]: Don't delete failed downloads #7796

Open
1 task done
66BA9Q-ME49 opened this issue Mar 16, 2025 · 1 comment
Open
1 task done

[enhancement]: Don't delete failed downloads #7796

66BA9Q-ME49 opened this issue Mar 16, 2025 · 1 comment
Labels
enhancement New feature or request

Comments

@66BA9Q-ME49
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Contact Details

No response

What should this feature add?

When a model fails to install or isn't recognized (due to an unsupported model format), Invoke should keep the downloaded files in a directory instead of deleting them.

Alternatives

No response

Additional Content

I tried downloading FLUX.1 [dev] using Invoke. It downloaded the model but failed to install it.

Image

[2025-03-16 22:15:08,695]::[ModelInstallService]::INFO --> Model download complete: black-forest-labs/FLUX.1-dev
[2025-03-16 22:15:08,702]::[ModelInstallService]::INFO --> Model install started: black-forest-labs/FLUX.1-dev
[2025-03-16 22:15:08,744]::[ModelInstallService]::ERROR --> Model install error: black-forest-labs/FLUX.1-dev
InvalidModelConfigException: Unknown base model for G:\03 - AI\Image Generation\InvokeAI\models\tmpinstall_az0bcd50\FLUX.1-dev

I thought that Invoke didn't support FLUX, so I went into the models folder to transfer the files to ComfyUI, but Invoke had already deleted the downloaded files.
Because of that, I waited several hours for nothing, and now I need to download 21GB of files again.

@66BA9Q-ME49 66BA9Q-ME49 added the enhancement New feature or request label Mar 16, 2025
@psychedelicious
Copy link
Collaborator

I think it's reasonable to retain models that downloaded successfully but failed to import. Probably should be behind a setting.

I'm happy to reviewing a PR that implements this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants