Skip to content

Fix Kornia pad import compatibility#498

Open
omribz156 wants to merge 1 commit into
Lightricks:masterfrom
omribz156:codex/kornia-pad-compat
Open

Fix Kornia pad import compatibility#498
omribz156 wants to merge 1 commit into
Lightricks:masterfrom
omribz156:codex/kornia-pad-compat

Conversation

@omribz156

Copy link
Copy Markdown

Fixes #496.

Summary

  • Removed the pad import from kornia.geometry.transform.pyramid.
  • Reused the existing torch.nn.functional as F import for the two padding calls in pyramid_blending.py.

This keeps the local reflect padding behavior without depending on Kornia exporting pad from its pyramid module, which is what breaks on newer Kornia installs. It should also help the related Cannot import name 'pad' report in #494.

Verification

  • python -m py_compile pyramid_blending.py
  • git diff --check

I did not run a full ComfyUI/Kornia runtime smoke locally because this machine does not have torch installed.

This was implemented with Codex assistance, with the patch kept focused and reviewed before opening.

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.

ComfyUI-LTXVideo does not work with newer versions of Kornia

1 participant