-
Notifications
You must be signed in to change notification settings - Fork 377
Missing model downloader #5929
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
Open
fragmede
wants to merge
5
commits into
Comfy-Org:main
Choose a base branch
from
fragmede:missing-model-downloader
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Missing model downloader #5929
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adds automatic download functionality to the Missing Models dialog. Features: - Automatically adds Download buttons to each missing model - Pre-configured URLs for popular models (SDXL, SD1.5, VAEs, LoRAs, etc.) - Real-time download progress shown in button (percentage) - Custom URL prompt for unknown models - Download All button for bulk downloads - TypeScript implementation with proper typing When the Missing Models dialog appears, users can now download missing models directly without manually searching and moving files. Requires backend API endpoints in ComfyUI server for download functionality.
- Automatically extract URLs from the dialog (no more prompting for URLs that are already shown) - Replace alert/prompt boxes with inline UI elements for better UX - Add inline status messages showing download progress and file sizes - Add inline URL input field when URL is not available - Show "Open URL" link when backend doesn't support downloads - Better error handling and status feedback The extension now: 1. Extracts existing URLs from the dialog HTML 2. Uses inline input fields instead of browser prompts 3. Shows real-time download progress with file sizes 4. Provides better visual feedback with status messages 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Remove /api prefix from model download endpoints - ComfyUI serves these endpoints at /models/download not /api/models/download - This fixes the 405 Method Not Allowed error 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Added data-download-button attribute to track existing buttons - Check for existing buttons before adding new ones - Prevents duplicate buttons from appearing in the UI 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Removed 'Download from provided URL' tooltip - Simplified button title to only show 'Click to enter URL' when needed - Cleaner UI without redundant text 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds an intuitive download system to the missing models dialog, allowing users to download required models directly from the ComfyUI interface with real-time progress tracking and download speed display.
Problem Solved
When users load workflows with missing models, they currently see a dialog listing the missing files but must:
This creates friction, especially for:
Solution
This PR enhances the missing models dialog with:
✨ User Experience Features
🎨 UI Implementation
🔒 Security Features
📁 Files Modified
src/extensions/core/missingModelsDownloader.ts
- Core implementationRelated PR
Also see comfyanonymous/ComfyUI#10224
Features in Detail
Individual Download Buttons
Each missing model gets a download button that:
Download All Button
Progress Display Format
Visual States
Screenshots
Before
After
Integration
Works with the companion backend PR that provides:
/models/download
API endpointsTesting
Tested scenarios:
Browser Compatibility
Tested on:
Performance
Related PR
See companion backend PR: ComfyUI#XXX for API implementation
Future Enhancements
Potential improvements for future iterations:
Note: This PR requires the companion backend PR to be merged for full functionality. The UI gracefully handles cases where the backend is not available.
┆Issue is synchronized with this Notion page by Unito