Windows CUDA ops and 5090 low-VRAM load path - #23
Draft
JonEcho wants to merge 1 commit into
Draft
Conversation
Build joyomni_ops on MSVC, skip VAE inductor compile on Windows, cap the allocator, and fall back from broken FP8 GEMM to bf16 dequant so 720p DiT load fits a 32 GB card.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
Windows load path for JoyAI-Video-Edit on an RTX 5090 (32 GB).
joyomni_opswith MSVC and CUDA 12.8 (deploy/build_joyomni_ops.ps1,deploy/install_vs2022_cpp.cmd,deploy/requirements-windows.txt).JOYOMNI_VAE_COMPILE=1.JOYOMNI_VRAM_CAP_GB.asyncioinstead ofuvlooponwin32.Stack
This is PR 1 of 2.
Child: JonEcho#1 (
file-restyle-loop-splice; base is this branch).Verification
git diff --stat origin/main...HEADshows 13 files, +241 / -65.code_rules_gate --stagedreported 25 introduced hits on this fork. Those hits match existing JoyAI style (imports inside functions,print(), largecreate_app). This contribution does not rewrite that style.JOYOMNI_VRAM_CAP_GB=26and VAE on CPU.Risks
build_joyomni_ops.ps1still assumes VS 2022 Communityvcvars64.batand a repo-root.venv.Follow-up
Merge stays with the maintainer. This pull request is draft.