Skip to content

[CI] Add LoRA inference tests#546

Merged
Edenzzzz merged 47 commits into
mainfrom
lora_tests
Jul 15, 2025
Merged

[CI] Add LoRA inference tests#546
Edenzzzz merged 47 commits into
mainfrom
lora_tests

Conversation

@Edenzzzz
Copy link
Copy Markdown
Collaborator

No description provided.

@Edenzzzz Edenzzzz temporarily deployed to runpod-runners June 24, 2025 03:40 — with GitHub Actions Inactive
@Edenzzzz Edenzzzz temporarily deployed to runpod-runners June 24, 2025 03:40 — with GitHub Actions Inactive
@Edenzzzz Edenzzzz temporarily deployed to runpod-runners June 24, 2025 03:40 — with GitHub Actions Inactive
@Edenzzzz Edenzzzz temporarily deployed to runpod-runners June 24, 2025 03:40 — with GitHub Actions Inactive
@Edenzzzz Edenzzzz temporarily deployed to runpod-runners June 24, 2025 03:40 — with GitHub Actions Inactive
@Edenzzzz Edenzzzz marked this pull request as draft June 30, 2025 19:35
output_dtype,
cast_forward_inputs=False)

set_mixed_precision_policy(master_dtype=default_dtype,
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't find master_dtype useful, and it also doesn't align with torch

@Edenzzzz Edenzzzz changed the title [CI] Add lora test [CI] Add lora tests Jul 5, 2025
@Edenzzzz Edenzzzz marked this pull request as ready for review July 5, 2025 01:03
@Edenzzzz Edenzzzz requested a review from kevin314 July 5, 2025 01:03
@Edenzzzz Edenzzzz temporarily deployed to runpod-runners July 5, 2025 01:07 — with GitHub Actions Inactive
@Edenzzzz Edenzzzz temporarily deployed to runpod-runners July 5, 2025 01:07 — with GitHub Actions Inactive
@Edenzzzz Edenzzzz changed the title [CI] Add lora inference tests [CI] Add LoRA inference tests Jul 8, 2025
@SolitaryThinker
Copy link
Copy Markdown
Collaborator

It looks like you didn't add the reference videos for lora similarity test. *.mp4 is in .gitignore so you'll need to do git add -f

image

Comment thread .pre-commit-config.yaml
@SolitaryThinker
Copy link
Copy Markdown
Collaborator

seems like there's still some errors in lora inference test. Also the SSIM test shouldn't time out (should finish in ~24 minutes or so)

@Edenzzzz
Copy link
Copy Markdown
Collaborator Author

There's still a bug with FSDP that doesn't allow re-register params for mutli-lora. Reaching out for a solution
pytorch/pytorch#157209

@Edenzzzz
Copy link
Copy Markdown
Collaborator Author

Added a note that currently must reload model to use multi-lora

@Edenzzzz
Copy link
Copy Markdown
Collaborator Author

@SolitaryThinker All tests passed. Let's merge this?

@Edenzzzz Edenzzzz requested a review from SolitaryThinker July 15, 2025 04:20
Comment thread fastvideo/utils/collect_env.py
@Edenzzzz Edenzzzz merged commit 7244a4b into main Jul 15, 2025
1 check was pending
@Edenzzzz Edenzzzz deleted the lora_tests branch July 15, 2025 20:06
@Edenzzzz Edenzzzz restored the lora_tests branch July 21, 2025 22:30
qimcis pushed a commit to qimcis/FastVideo that referenced this pull request Oct 30, 2025
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.

2 participants