Skip to content

Automatic model downloading and building#14

Open
reaperhammer wants to merge 8 commits intoyuvraj108c:masterfrom
reaperhammer:master
Open

Automatic model downloading and building#14
reaperhammer wants to merge 8 commits intoyuvraj108c:masterfrom
reaperhammer:master

Conversation

@reaperhammer
Copy link
Copy Markdown

Add automatic model download and build, with the 2 node method, same as ComfyUI-Upscaler-Tensorrt
Add FP32 option
Update readme with new versions of supported Python, Cuda and Tensorrt that I'm using, in WSL
Update readme with new usage instructions

fix: add export_trt.py to sys.path to fix it for use with comfyui's python_embeded/python.exe
Update the Python and Cuda version images as it works with these later versions that I'm using
more readme updates
Reordered and uncommented dependencies in requirements.txt
vitosans added a commit to ComfyAssets/ComfyUI-Rife-Tensorrt that referenced this pull request Mar 12, 2026
…view

- Add HTTP error checking and atomic writes to download_file()
- Check engine.build() return value and clean up partial .trt files on failure
- Use cudaGraphExecDestroy (not cudaGraphDestroy) for cudaGraphExec_t handles
- Add hasattr guards in Engine.__del__ to handle partial construction
vitosans added a commit to ComfyAssets/ComfyUI-Rife-Tensorrt that referenced this pull request Mar 12, 2026
Fix error handling and CUDA cleanup from PR yuvraj108c#14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant