-
Notifications
You must be signed in to change notification settings - Fork 144
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
add load 3d node support #1563
add load 3d node support #1563
Conversation
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/[email protected] |
currently, the two browers tests failed but I am not sure it is caused by my changes, I checked another recent PR, like #1537, it also has such issues @huchenlei |
Sorry about that. It was introduced by accident. Please rebase your branch. The playwright test failure was fixed in #1561 |
c51821d
to
9c98c6a
Compare
Done |
/home/runner/work/ComfyUI_frontend/ComfyUI_frontend/src/stores/coreSettings.ts |
This is exactly what I was looking for for a while! How can I use this node?? |
hi @yovsac , as you see, the FE code already merged, |
Thank you for your quick reply. Im using Comfyui portable. Can you tell me how I can copy the code? |
@yovsac I just realize coping code to your local env might not a good idea if you are using comfyui portable version, because I am not sure it is already included the FE code here.
Still, this method may not work as your local env might not included the latest FE code. |
Thank you very much. I’ll try that!! |
@yovsac maybe the best idea is, could you help to go the core PR comfyanonymous/ComfyUI#5564, leave your idea and may push it merge quickly? |
I left a message. Hope they hurry up! |
Hi,
I implemented a node to load local 3d models and output IMAGE for furthur process, such as controlnet, as reference image.
here is a demo video:
https://github.com/user-attachments/assets/703b12ed-9e60-42df-9de2-31fa1df48ce7
Need comfyanonymous/ComfyUI#5564 as BE