-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
Why have a venv if stab matrix won't install python packages to it? #423
Comments
Our package install process and the python package manager menu should both install into the package's venv. Can you describe what behavior you're observing and what behavior you expect. |
Dependencies for imported nodes constantly get installed to the standalone
python rather than the venv.
…On Mon., 29 Jan. 2024, 3:16 am Ionite, ***@***.***> wrote:
Our package install process and the python package manager menu should
both install into the package's venv. Can you describe what behavior you're
observing and what behavior you expect.
—
Reply to this email directly, view it on GitHub
<#423 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADLY5PFEJQHNE6EQZVB7IJ3YQ2BYRAVCNFSM6AAAAABCOC6UISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGY3DKMZSHA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
How did you install the imported nodes, is it the builtin extensions manager or ComfyUI manager, if so did you install ComfyUI Manager the normal way or the portable way for embedded python? The latter (portable / embedded) method only works with embedded ComfyUI and not Stability Matrix. Following their normal instructions for git cloning to extra nodes folder should work fine with the venv. |
All three, standalone, portable and stability matrix. All three run into
issues of not installing packages to the correct python location or venv.
Makes it even more difficult when there's version discrepancies for
dependencies based on which packages are needed for what components.
…On Mon., 5 Feb. 2024, 6:52 am Ionite, ***@***.***> wrote:
How did you install the imported nodes, is it the builtin extensions
manager or ComfyUI manager, if so did you install ComfyUI Manager the
normal way or the portable way for embedded python? The latter (portable /
embedded) method only works with embedded ComfyUI and not Stability Matrix.
Following their normal instructions for git cloning to extra nodes folder
should work fine with the venv.
—
Reply to this email directly, view it on GitHub
<#423 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADLY5PCFWYJLKEKBSTRCBLLYR7YH7AVCNFSM6AAAAABCOC6UISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRVHEYTANZTHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Could you give a list of steps you took, and what you observed as a result of it not installing into the venv? This shouldn't occur with ComfyUI installs in Stability Matrix. |
Why have a venv if stab matrix won't install python packages to it?
The text was updated successfully, but these errors were encountered: