Skip to content

Conversation

victorges
Copy link
Member

Pin required preprocessor packages and update the TensorRT installer to use stable NVIDIA PyPI builds.

The packages controlnet-aux, mediapipe, and insightface were previously either missing from requirements or treated as optional, leading to installation issues and unexpected behavior. They are now explicitly required and pinned to known-good versions. The TensorRT installer was updated to use stable, officially released versions from NVIDIA's PyPI, improving reliability and addressing a long-standing TODO.


Open in Cursor Open in Web

Co-authored-by: victorgelias <victorgelias@gmail.com>
Copy link

cursor bot commented Sep 23, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

cursoragent and others added 5 commits September 23, 2025 15:53
Co-authored-by: victorgelias <victorgelias@gmail.com>
Co-authored-by: victorgelias <victorgelias@gmail.com>
Co-authored-by: victorgelias <victorgelias@gmail.com>
Co-authored-by: victorgelias <victorgelias@gmail.com>
Co-authored-by: victorgelias <victorgelias@gmail.com>
Co-authored-by: victorgelias <victorgelias@gmail.com>
Copy link
Member Author

@victorges victorges left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

almost there

cursoragent and others added 2 commits September 23, 2025 17:57
Co-authored-by: victorgelias <victorgelias@gmail.com>
Co-authored-by: victorgelias <victorgelias@gmail.com>
@victorges victorges marked this pull request as ready for review September 23, 2025 18:27
@victorges victorges changed the title Update requirements and install tensorrt script install: Update setup and install-tensorrt scripts Sep 24, 2025
@BuffMcBigHuge
Copy link
Collaborator

Added some fixed deps in requirements.txt for for Windows and realtime-img2img from here.

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.

4 participants