nodes/magicwang1111-Comfyui-Kling-Wrapper
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Make preview video save by default

Browse Source
This commit is contained in:
magicwang1111
2026-04-28 10:28:24 +08:00
parent a416aa3682
commit dbd985dfd8
12 changed files with 17 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files
examples/02_comfyui_kling_wrapper_text2video_v3_intelligence.json
+1 -2
View File
@@ -88,8 +88,7 @@
"Node name for S&R": "Comfyui-Kling-Wrapper Preview Video"
},
"widgets_values": [
"Comfyui-Kling-Wrapper",
true
"Comfyui-Kling-Wrapper"
]
}
],
examples/03_comfyui_kling_wrapper_image2video_v3.json
+1 -2
View File
@@ -137,8 +137,7 @@
"Node name for S&R": "Comfyui-Kling-Wrapper Preview Video"
},
"widgets_values": [
"Comfyui-Kling-Wrapper",
true
"Comfyui-Kling-Wrapper"
]
}
],
examples/04_comfyui_kling_wrapper_multi_images_to_video_v3.json
+1 -2
View File
@@ -208,8 +208,7 @@
"Node name for S&R": "Comfyui-Kling-Wrapper Preview Video"
},
"widgets_values": [
"Comfyui-Kling-Wrapper",
true
"Comfyui-Kling-Wrapper"
]
}
],
examples/05_comfyui_kling_wrapper_text2video_v26_sound.json
+1 -2
View File
@@ -88,8 +88,7 @@
"Node name for S&R": "Comfyui-Kling-Wrapper Preview Video"
},
"widgets_values": [
"Comfyui-Kling-Wrapper",
true
"Comfyui-Kling-Wrapper"
]
}
],
examples/06_comfyui_kling_wrapper_advanced_element_subject_to_image2video.json
+1 -2
View File
@@ -396,8 +396,7 @@
"Node name for S&R": "Comfyui-Kling-Wrapper Preview Video"
},
"widgets_values": [
"Comfyui-Kling-Wrapper",
true
"Comfyui-Kling-Wrapper"
]
}
],
examples/07_comfyui_kling_wrapper_motion_control_v26.json
+1 -2
View File
@@ -165,8 +165,7 @@
"Node name for S&R": "Comfyui-Kling-Wrapper Preview Video"
},
"widgets_values": [
"Comfyui-Kling-Wrapper",
true
"Comfyui-Kling-Wrapper"
]
}
],
examples/10_comfyui_kling_wrapper_video_extend_chain.json
+1 -2
View File
@@ -131,8 +131,7 @@
"Node name for S&R": "Comfyui-Kling-Wrapper Preview Video"
},
"widgets_values": [
"Comfyui-Kling-Wrapper",
true
"Comfyui-Kling-Wrapper"
]
}
],
examples/13_comfyui_kling_wrapper_lip_sync_from_text.json
+1 -2
View File
@@ -170,8 +170,7 @@
"Node name for S&R": "Comfyui-Kling-Wrapper Preview Video"
},
"widgets_values": [
"Comfyui-Kling-Wrapper",
true
"Comfyui-Kling-Wrapper"
]
}
],
examples/14_comfyui_kling_wrapper_effects_single_image.json
+1 -2
View File
@@ -128,8 +128,7 @@
"Node name for S&R": "Comfyui-Kling-Wrapper Preview Video"
},
"widgets_values": [
"Comfyui-Kling-Wrapper",
true
"Comfyui-Kling-Wrapper"
]
}
],
examples/generate_examples.py
+1 -1
View File
@@ -170,7 +170,7 @@ def preview_video_node(node_id, pos, order, link_id):
pos,
[430, 820],
order,
[NODE_PREFIX, True],
[NODE_PREFIX],
inputs=[input_slot("video_url", "STRING", link_id)],
outputs=[output_slot("file_path", "STRING", slot_index=0)],
)
py/nodes.py
+1 -2
View File
@@ -1523,7 +1523,6 @@ class PreviewVideo:
"required": {
"video_url": ("STRING", {"forceInput": True}),
"filename_prefix": ("STRING", {"default": DEFAULT_FILENAME_PREFIX}),
"save_output": ("BOOLEAN", {"default": True}),
}
}
@@ -1534,7 +1533,7 @@ class PreviewVideo:
RETURN_TYPES = ("STRING",)
RETURN_NAMES = ("file_path",)
def run(self, video_url, filename_prefix, save_output):
def run(self, video_url, filename_prefix, save_output=True):
if not save_output:
return {"ui": {"video_url": [video_url]}, "result": ('',)}
tests/test_backend_config.py
+6 -4
View File
@@ -142,7 +142,6 @@ class BackendConfigTests(unittest.TestCase):
result = kling_nodes.PreviewVideo().run(
"https://example.com/video.mp4",
"Comfyui-Kling-Wrapper",
True,
)
self.assertEqual(
@@ -154,6 +153,11 @@ class BackendConfigTests(unittest.TestCase):
"Comfyui-Kling-Wrapper_00001_.mp4",
)
def test_preview_video_save_output_is_not_user_toggleable(self):
required = kling_nodes.PreviewVideo.INPUT_TYPES()["required"]
self.assertNotIn("save_output", required)
def test_preview_video_registers_saved_file_as_asset(self):
with tempfile.TemporaryDirectory() as tmpdir:
tmpdir_path = Path(tmpdir)
@@ -169,7 +173,6 @@ class BackendConfigTests(unittest.TestCase):
result = kling_nodes.PreviewVideo().run(
"https://example.com/video.mp4",
"Comfyui-Kling-Wrapper",
True,
)
register_mock.assert_called_once_with(result["result"][0])
@@ -189,7 +192,6 @@ class BackendConfigTests(unittest.TestCase):
result = kling_nodes.PreviewVideo().run(
"https://example.com/video.mp4",
"Comfyui-Kling-Wrapper",
True,
)
self.assertTrue(Path(result["result"][0]).is_file())
@@ -199,7 +201,7 @@ class BackendConfigTests(unittest.TestCase):
ValueError,
"empty video_url",
):
kling_nodes.PreviewVideo().run("", "Comfyui-Kling-Wrapper", True)
kling_nodes.PreviewVideo().run("", "Comfyui-Kling-Wrapper")
def test_upload_file_to_tmpfiles_retries_transient_ssl_error(self):
with tempfile.TemporaryDirectory() as tmpdir: