Skip to content
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

【Hackathon 5th】PaddleMIX ppdiffusers中升级embeddings代码 #260 #333

Closed
wants to merge 1 commit into from
Closed

Conversation

zoey333
Copy link

@zoey333 zoey333 commented Nov 30, 2023

PaddleMIX ppdiffusers中升级embeddings代码 #260
对比ppdiffusers (v0.19.3)和diffusers (v0.21.1)的代码差异,新版本增加了两个类

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


zoey333 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@zoey333
Copy link
Author

zoey333 commented Dec 1, 2023

@shiyutang @JunnYu



#v0.21.1新增
class FourierEmbedder(nn.Module):
Copy link
Member

Choose a reason for hiding this comment

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

这些都是torch的代码?

Copy link
Author

Choose a reason for hiding this comment

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

对比两个版本的差异,569行后新增的代码,都是新版本增加的。都是torch的代码是指?

Copy link
Member

Choose a reason for hiding this comment

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

对比差异,我们需要添加paddle实现的代码,而不是torch的代码

Copy link
Author

Choose a reason for hiding this comment

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

明白了,有相关教程吗?我看着改下

@shiyutang shiyutang changed the title PaddleMIX ppdiffusers中升级embeddings代码 #260 【Hackathon 5th】PaddleMIX ppdiffusers中升级embeddings代码 #260 Dec 13, 2023
@zoey333 zoey333 closed this Dec 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants