Skip to content

Add video-to-video #1469

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Jun 24, 2025
Merged

Add video-to-video #1469

merged 14 commits into from
Jun 24, 2025

Conversation

merveenoyan
Copy link
Contributor

Copy link
Member

@Vaibhavs10 Vaibhavs10 left a comment

Choose a reason for hiding this comment

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

cc: @linoytsaban - since you wanted to open a PR for this!

@@ -111,6 +111,7 @@ export type {
} from "./zero-shot-object-detection/inference.js";

import type { ModelLibraryKey } from "../model-libraries.js";
import { get } from "http";
Copy link
Member

Choose a reason for hiding this comment

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

is this supposed to be here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

idk where it came from lol thx for pointing out, fixed

Copy link
Member

@Vaibhavs10 Vaibhavs10 left a comment

Choose a reason for hiding this comment

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

Looks good to me! Do we have a list of models where this would already apply?

IIRC @linoytsaban had 10-12 models she was tracking!

@merveenoyan
Copy link
Contributor Author

@Vaibhavs10 this yields 62 models https://huggingface.co/search/full-text?q=video-to-video&type=model I'll open PRs

@merveenoyan
Copy link
Contributor Author

@Vaibhavs10 I opened the PRs, we can merge when some are merged

@merveenoyan
Copy link
Contributor Author

merged main to this branch, it broke one test ^_^

Copy link
Member

@Vaibhavs10 Vaibhavs10 left a comment

Choose a reason for hiding this comment

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

Broken test is unrelated to the PR, let's wait for more than half of PRs to be merged before merging this please.

Excited for this! 🤗

@sayakpaul
Copy link
Member

Cool work!

@linoytsaban
Copy link
Contributor

awesome 🤗 opened PRs for the VACE models too
(https://huggingface.co/Wan-AI/Wan2.1-VACE-14B and the others in: https://huggingface.co/collections/ali-vilab/vace-67eca186ff3e3564726aff38)

@NielsRogge
Copy link
Contributor

Could also be relevant for datasets, e.g. Gemini added it here: https://huggingface.co/datasets/Iceclear/SeedVR_VideoDemos.

@merveenoyan
Copy link
Contributor Author

opened PRs for SeedVR. the moon one can't be merged without this one being merged (it will fail)

this one errors come from mcp client and not this PR, can someone review and we can merge

@merveenoyan merveenoyan requested a review from pcuenca June 20, 2025 16:36
@merveenoyan merveenoyan requested a review from Vaibhavs10 June 23, 2025 16:42
@merveenoyan
Copy link
Contributor Author

I think node and browser error isn't related to this PR

Copy link
Member

@Vaibhavs10 Vaibhavs10 left a comment

Choose a reason for hiding this comment

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

CI errors are unrelated feel free to merge

@merveenoyan merveenoyan merged commit 71495e2 into main Jun 24, 2025
3 of 5 checks passed
@merveenoyan merveenoyan deleted the add-video-to-video branch June 24, 2025 11:42
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.

6 participants