Comfy-Org/ComfyUI_Nvidia_VFX_Nodes
1
0
Fork 0
mirror of https://github.com/Comfy-Org/ComfyUI_Nvidia_VFX_Nodes.git synced 2026-11-04 23:37:30 +00:00
Code Issues 31 Projects Packages Wiki Activity

[PR #15] [CLOSED] feat: split into RTXImageSuperResolution and RTXVideoSuperResolution #37

New issue
Closed
opened 2026-10-26 22:28:17 +00:00 by bsakaguchi · 0 comments
bsakaguchi commented 2026-10-26 22:28:17 +00:00
Owner
Copy link

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/ComfyUI_Nvidia_VFX_Nodes/pull/15
Author: @deepme987
Created: 3/13/2026
Status: ❌ Closed

Base: main ← Head: split-image-video-nodes

📝 Commits (1)

  • 30a53c0 feat: split into RTXImageSuperResolution and RTXVideoSuperResolution

📊 Changes

1 file changed (+108 additions, -39 deletions)

View changed files

📝 __init__.py (+108 -39)

📄 Description

Summary

  • Split the single RTXVideoSuperResolution node into two separate nodes:
    • RTXImageSuperResolution: Upscales a single image. Simpler execution path without batch processing.
    • RTXVideoSuperResolution: Upscales a batch of video frames with configurable batch sizing for memory management.
  • Shared logic (dimension resolution, quality mapping) extracted into module-level helpers
  • Both nodes retain the same resize type options (scale by multiplier / target dimensions) and quality levels (LOW/MEDIUM/HIGH/ULTRA)

Motivation

Having separate image and video nodes allows cloud environments to selectively enable the image upscaler while disabling the video upscaler (which requires significantly more system RAM for batch processing). This unblocks cloud deployment of RTX Super Resolution for single-image use cases.

Test plan

  • Verify RTXImageSuperResolution upscales a single image correctly (scale by + target dimensions)
  • Verify RTXVideoSuperResolution still works identically to the original node for video frame batches
  • Verify both nodes appear in ComfyUI's node list under image/upscaling

🤖 Generated with Claude Code

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/ComfyUI_Nvidia_VFX_Nodes/pull/15 **Author:** [@deepme987](https://github.com/deepme987) **Created:** 3/13/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `split-image-video-nodes` --- ### 📝 Commits (1) - [`30a53c0`](https://github.com/Comfy-Org/ComfyUI_Nvidia_VFX_Nodes/commit/30a53c054d0bec4b76bd37981b1abe62d85012e4) feat: split into RTXImageSuperResolution and RTXVideoSuperResolution ### 📊 Changes **1 file changed** (+108 additions, -39 deletions) <details> <summary>View changed files</summary> 📝 `__init__.py` (+108 -39) </details> ### 📄 Description ## Summary - Split the single `RTXVideoSuperResolution` node into two separate nodes: - **`RTXImageSuperResolution`**: Upscales a single image. Simpler execution path without batch processing. - **`RTXVideoSuperResolution`**: Upscales a batch of video frames with configurable batch sizing for memory management. - Shared logic (dimension resolution, quality mapping) extracted into module-level helpers - Both nodes retain the same resize type options (scale by multiplier / target dimensions) and quality levels (LOW/MEDIUM/HIGH/ULTRA) ## Motivation Having separate image and video nodes allows cloud environments to selectively enable the image upscaler while disabling the video upscaler (which requires significantly more system RAM for batch processing). This unblocks cloud deployment of RTX Super Resolution for single-image use cases. ## Test plan - [ ] Verify `RTXImageSuperResolution` upscales a single image correctly (scale by + target dimensions) - [ ] Verify `RTXVideoSuperResolution` still works identically to the original node for video frame batches - [ ] Verify both nodes appear in ComfyUI's node list under `image/upscaling` 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-26 22:28:17 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
alexis/rename-true-hdr-to-video-hdr
Revert-node-name
alexis/add-long-video-true-hdr-workflow
alexis/add-node-thumbnails-readme
alexis/add-video-upscale-thumbnail
alexis/update-workflow-thumbnails
alexis/update-workflows
Update-README
alexis/update-tru-hdr-workflow
alexis/update-vfx-nodes
alexis/update-pyproject
add-publish-workflow-and-pyproject
No results found.
Labels
Clear labels   
pull-request
Mirrored from GitHub Pull Request

No labels
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Comfy-Org/ComfyUI_Nvidia_VFX_Nodes#37
No description provided.