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

Add Replicate demo and API #334

Merged
merged 10 commits into from
Sep 26, 2024
Merged

Add Replicate demo and API #334

merged 10 commits into from
Sep 26, 2024

Conversation

chenxwh
Copy link
Contributor

@chenxwh chenxwh commented Sep 22, 2024

Hi @zRzRzRzRzRzRzR,

Great work on CogVideoX!

This pull request makes it possible to run CogVideoX on Replicate (https://replicate.com/chenxwh/cogvideox-t2v, https://replicate.com/chenxwh/cogvideox-i2v). Also we'd like to transfer the demo pages to THUDM so you can make modifications easily, and happy to help maintain/integrate the upcoming changes and improving the demo :)

Copy link
Member

@zRzRzRzRzRzRzR zRzRzRzRzRzRzR left a comment

Choose a reason for hiding this comment

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

I think this is a very good tool, placing it at https://github.com/THUDM/CogVideo/tree/main/tools and creating a separate folder is a better choice. Would you be willing to make such a modification?

Copy link
Member

Choose a reason for hiding this comment

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

I think it should be placed in tools, not in the readme on the homepage. We can add a news item in news. Can you modify both the Chinese and Japanese versions together?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Modified the main Readme

@chenxwh
Copy link
Contributor Author

chenxwh commented Sep 25, 2024

@zRzRzRzRzRzRzR thanks for the suggestion, I have made the corresponding modifications :)

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