6 Commits
Author SHA1 Message Date
PozzettiAndrea d62a86fbe7 better controls 2026-06-07 09:00:00 +02:00
PozzettiAndrea dabd3a5527 4 workflows 2026-06-07 09:00:00 +02:00
PozzettiAndrea 006259f0b2 ms 2026-05-16 16:12:57 +02:00
PozzettiAndrea 93bc2e3775 LoadPLY: switch to LoadImage idiom — bare filenames + get_annotated_filepath
Previous design scanned input/ AND output/ and stored prefixed combo
entries like 'input/apple.ply'. When comfy-test ran the workflow, the
saved value 'input/apple.ply' didn't match anything the node could
resolve, and VALIDATE_INPUTS rejected the prompt:

  Custom validation failed for node: ply_file -
  LoadPLY: 'input/apple.ply' is not a known PLY in input/ or output/

Refactor to mirror ComfyUI's stock LoadImage node:
- INPUT_TYPES lists ONLY input/, bare filenames (no folder prefix)
- IS_CHANGED uses folder_paths.get_annotated_filepath(name)
- VALIDATE_INPUTS uses folder_paths.exists_annotated_filepath(name)
- load() returns folder_paths.get_annotated_filepath(name)

`get_annotated_filepath` understands `[input]/name`, `[temp]/name`,
and `[output]/name` annotations transparently, so power users can
type those into the combo if they need output/ access — same as
LoadImage. The default UX is "pick from input/".

Output-side PLYs from upstream nodes (GaussianMerge, GaussianExport,
LiToExportPLY) are wired directly into PreviewGaussians via the
STRING output — no LoadPLY needed.

workflows/merge_splats.json: rewrite widgets_values from
"input/apple.ply" -> "apple.ply" to match the new bare-filename
convention.
2026-05-16 16:06:21 +02:00
PozzettiAndrea eee3bee6ba new workflow 2026-05-16 15:40:07 +02:00
PozzettiAndrea 11323079d2 added workflow 2026-05-16 15:08:00 +02:00