Skip to content

Use Comfy3D-maintained geometry addon forks - #7

Open
michaelgold wants to merge 26 commits into
mainfrom
fix/pozzetti-addon-pins
Open

Use Comfy3D-maintained geometry addon forks#7
michaelgold wants to merge 26 commits into
mainfrom
fix/pozzetti-addon-pins

Conversation

@michaelgold

@michaelgold michaelgold commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • switches UniRig install from PozzettiAndrea/ComfyUI-UniRig stale commit to michaelgold/ComfyUI-UniRig commit c68895bd9f4ad44451bec335be97f0b98ccddabd
  • switches GeometryPack to michaelgold/ComfyUI-GeometryPack commit 8229aa65ec026b4b87802e4ac3b28022c8dd628b
  • preserves Blender/bpy in the Comfy3D runtime instead of custom-node comfy-env/pixi/conda bootstrapping
  • adds Remesher workflow node-registration smoke against remesher/remesher
  • adds CUDA 13 Remesher CLI avatar smoke: text-to-image -> image-to-GLB -> direct MIA rigged GLB
  • caches Qwen, Trellis/DINO, and MIA model files in the mounted model cache
  • uses isolated subprocess bpy worker for MIA GLB export
  • bakes armature object scale before direct GLB export so importers do not show 100x bones

Validation

  • Local py_compile / YAML parse / git diff checks pass
  • GitHub Actions CUDA 12.8 and CUDA 13 matrix was green before the scale-bake pin update
  • Manual mghavn smoke passed with CI/CD model preflight mirrored
  • Manual bpy validation imported the corrected GLB, confirmed armature scale [1,1,1], found 53 bones, posed LeftArm/LeftForeArm, rendered rest and posed frames, and exported posed GLB

Notes

  • Full Remesher avatar smoke runs only on CUDA 13.0 for now to control CI time
  • HF_TOKEN is required for gated DINOv3 model cache preflight

Copilot AI review requested due to automatic review settings July 20, 2026 17:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Docker image build to install Comfy3D-maintained forks of UniRig and GeometryPack, aiming to avoid failures caused by upstream addon commits that depend on incompatible comfy-env/pixi behavior.

Changes:

  • Switch UniRig install source from PozzettiAndrea/ComfyUI-UniRig to michaelgold/ComfyUI-UniRig (comfy3d/no-comfy-env).
  • Switch GeometryPack install source from PozzettiAndrea/ComfyUI-GeometryPack to michaelgold/ComfyUI-GeometryPack (comfy3d/no-comfy-env).
  • Keep installation behavior independent of upstream comfy-env/pixi assumptions for these addons.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Docker/Dockerfile Outdated
Comment on lines +312 to +314
RUN python /app/utils/node_install.py https://github.com/michaelgold/ComfyUI-UniRig --version comfy3d/no-comfy-env --no-build-isolation

RUN python /app/utils/node_install.py https://github.com/PozzettiAndrea/ComfyUI-GeometryPack --version 3cb49fbd7ba911c10a72fc767926a029f97798ab --no-build-isolation
RUN python /app/utils/node_install.py https://github.com/michaelgold/ComfyUI-GeometryPack --version comfy3d/no-comfy-env --no-build-isolation
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.

2 participants