UI→API conversion sends LOAD_3D viewport_state as "" where the frontend sends the viewport camera

Đang mở
  • Có gì không ổn?
#1,126 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
53/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
cli

Hướng nghiên cứu

Start with convert_ui_to_api and trace how it serializes the LOAD_3D viewport_state widget; compare the result with the frontend’s graphToPrompt output described in the issue. Check the bundled templates 3d_pixal3d_multi_views and 3d_pixal3d_trellis2_image_to_model, then verify that conversion preserves the viewport state needed by execute_save_3d_advanced in comfy_extras/nodes_save_3d.py. Done means the converted prompt provides the expected camera information, including for downstream wiring.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

UI→API conversion sends viewport_state as the saved "", while the frontend's graphToPrompt sends the live 3D viewport state (camera, model info). Found by converting the bundled 3D templates with convert_ui_to_api (the comfy run --workflow path) and comparing each prompt with the frontend's app.graphToPrompt().output for the same graph.

Versions: comfy-cli 54732c6 (main), ComfyUI v0.44.2 on CPU, comfyui-frontend-package 1.57.6, comfyui-workflow-templates 0.11.103.

Where: Preview3DAdvanced and Save3DAdvanced (input viewport_state, type LOAD_3D, required). Templates 3d_pixal3d_multi_views (nodes 246, 323, 372) and 3d_pixal3d_trellis2_image_to_model (nodes 246, 322, 323). The saved node has widgets_values: ["", 1024, 1024].

  • comfy-cli: "viewport_state": ""
  • frontend: "viewport_state": {"image": "", "mask": "", "normal": "", "camera_info": {"position": {"x": 10, "y": 10, "z": 10.000000000000002}, "target": {"x": 0, "y": 0, "z": 0}, "zoom": 1, "cameraType": "perspective", ...}, "recording": "", "model_3d_info": []}

Effect: execute_save_3d_advanced (comfy_extras/nodes_save_3d.py) turns a non-dict viewport_state into {}, so the node's camera_info output, and the camera in its Saved3DModels UI result, are None instead of the viewport camera. In these two templates nothing is wired to those outputs, so the generated model is unaffected; a graph that wires camera_info downstream would get None.

The frontend builds this value at serialisation time from the live viewport rather than from widgets_values, so matching it exactly may need a default camera on the converter side, or the server treating "" as "default camera". Related: #893 (widget types without a slot), #1077 (converter convergence).

Ngôn ngữ chính
Python
Star
1k
Fork
176
Merge trung bình
1 ngày 11 giờ
Pull request đã merge (30 ngày)
87

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Comfy-Org/comfy-cli

Tất cả issue của Comfy-Org/comfy-cli

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.