-
Notifications
You must be signed in to change notification settings - Fork 9.2k
Support Iluvatar CoreX #8585
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
base: master
Are you sure you want to change the base?
Support Iluvatar CoreX #8585
Conversation
46d9466
to
da50a8e
Compare
fa8a063
to
c795d23
Compare
Hello @comfyanonymous Is this PR still under review? Let me know if there is anything else I need to do. From the feedback from the community, many users expect to use ComfyUI on Iluvatar CoreX GPU. I submitted this PR and hope it can be merged as soon as possible. Thanks! Here I have rebased the new master, and have tested it, the result showed it worked well. The test logs and images are shown below. If you mind we modify the cuda malloc to adapt Iluvatar CoreX GPU, we can also revert the c795d23 and launch ComfyUI by running PTAL, Thanks!
|
c795d23
to
7d9d546
Compare
cuda_malloc.py
Outdated
@@ -50,7 +50,33 @@ def enum_display_devices(): | |||
"GeForce GTX 1650", "GeForce GTX 1630", "Tesla M4", "Tesla M6", "Tesla M10", "Tesla M40", "Tesla M60" | |||
} | |||
|
|||
def _load_torch_submodule(filename): | |||
"""Helper to load and check a submodule from torch's installation""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of doing this can't you just check if the computer has an iluvatar device?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @comfyanonymous
Thank you for your time and support.
We’ve looked into the issue and would like to propose two possible approaches to address it:
- Modify
cuda_malloc.py
to detect Iluvatar GPU names by usingsubprocess.check_output(['ixsmi', '-L'])
, following a pattern similar to how NVIDIA GPUs are currently detected. - As an alternative, we could leave
cuda_malloc.py
unchanged and instead update the README to include instructions for launching ComfyUI with the command:python main.py --disable-cuda-malloc
.
Both options seem viable, but we’d really appreciate your thoughts on which direction would be more suitable for the project.
Thanks again for your guidance
4b6d9a5
to
dda8034
Compare
hello @comfyanonymous , we have updated the README to include instructions for launching ComfyUI with the command: python main.py --disable-cuda-malloc. PTAL, Thanks! |
46ebfd4
to
b0ac606
Compare
@comfyanonymous @ltdrdata PTAL. |
Co-authored-by: mingjiang.li <[email protected]>
2299df3
to
3280340
Compare
@comfyanonymous we have rebase the new master, and because of this #9031, we do not need add PTAL, thanks! |
close #8584
The PR was validated on Iluvatar CoreX GPUs. We need to install Iluvatar Corex Toolkit first. Then run:
We use the sd_xl_base_1.0 model and get the default workflow's results as below: