Skip to content

Update the reference for text_to_video.md#5706

Merged
patrickvonplaten merged 7 commits intohuggingface:mainfrom
JacobYuan7:patch-1
Nov 13, 2023
Merged

Update the reference for text_to_video.md#5706
patrickvonplaten merged 7 commits intohuggingface:mainfrom
JacobYuan7:patch-1

Conversation

@JacobYuan7
Copy link
Copy Markdown
Contributor

The original reference (VideoFusion) might be misleading. VideoFusion is not open-sourced. I am the co-first author of ModelScopeT2V. I change the referred paper to the right one.

What does this PR do?

Fixes # (issue)

Before submitting

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

The original reference (VideoFusion) might be misleading. VideoFusion is not open-sourced. I am the co-first author of ModelScopeT2V. I change the referred paper to the right one.
@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

HuggingFaceDocBuilderDev commented Nov 8, 2023

The documentation is not available anymore as the PR was closed or merged.

Copy link
Copy Markdown
Member

@stevhliu stevhliu left a comment

Choose a reason for hiding this comment

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

Ah, very sorry for the incorrect reference! 😅

Comment thread docs/source/en/api/pipelines/text_to_video.md
Comment thread docs/source/en/api/pipelines/text_to_video.md Outdated
@JacobYuan7
Copy link
Copy Markdown
Contributor Author

@stevhliu Many thanks for the prompt response! But not sure whether this edit has been merged to the main branch.

yiyixuxu and others added 2 commits November 9, 2023 11:39
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

@JacobYuan7 JacobYuan7 left a comment

Choose a reason for hiding this comment

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

I am good with this edit. Thanks.

@patrickvonplaten
Copy link
Copy Markdown
Contributor

Thanks for the correction @JacobYuan7

@patrickvonplaten patrickvonplaten merged commit 4d3b4e0 into huggingface:main Nov 13, 2023
@JacobYuan7
Copy link
Copy Markdown
Contributor Author

@patrickvonplaten Pleasure.

@JacobYuan7
Copy link
Copy Markdown
Contributor Author

@stevhliu May I know when this change will be shown in the default Stable version (V0.23.0) of Diffusers? In this version, this reference information is still the old one.

@stevhliu
Copy link
Copy Markdown
Member

Hi, this change is currently live on the main version and it'll be shown in the stable version when we do the next release.

cc @patrickvonplaten do you know when this will be?

AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
* Update the reference for text_to_video.md

The original reference (VideoFusion) might be misleading. VideoFusion is not open-sourced. I am the co-first author of ModelScopeT2V. I change the referred paper to the right one.

* Update docs/source/en/api/pipelines/text_to_video.md

Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>

---------

Co-authored-by: YiYi Xu <yixu310@gmail.com>
Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
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.

5 participants