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

Contributing to task page for any-to-any #1011

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

auroraalabs
Copy link

No description provided.

@auroraalabs
Copy link
Author

Hii can anyone help push this along? :
2 workflows awaiting approval

@SBrandeis @gary149 @Wauplin @pcuenca @ngxson @merveenoyan

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.

Hi @auroraalabs - Let's wait for @merveenoyan to review and approve this. This PR is already in her notifications so should be sorted soon.

And thanks for opening a PR 🤗

Copy link
Contributor

@merveenoyan merveenoyan left a comment

Choose a reason for hiding this comment

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

Thanks a ton for contributing! 💗

packages/tasks/src/tasks/any-to-any/data.ts Outdated Show resolved Hide resolved
packages/tasks/src/tasks/any-to-any/data.ts Show resolved Hide resolved
{
description: "Janus is a versatile multimodal framework that decouples visual encoding for superior understanding and generation across tasks.",
description:
"Janus is a versatile multimodal framework that decouples visual encoding for superior understanding and generation across tasks.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment applies here

packages/tasks/src/tasks/any-to-any/data.ts Outdated Show resolved Hide resolved
id: "PKU-Alignment/align-anything-400k",
},],
},
],
demo: {
inputs: [],
Copy link
Contributor

Choose a reason for hiding this comment

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

would you like to add a demo as well? maybe image/text input to image/text output

Copy link
Author

Choose a reason for hiding this comment

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

Added one!

@merveenoyan
Copy link
Contributor

@auroraalabs BTW since you asked to push this along, I need to mention that we will not be deploying until the task page is complete, including about.md part. I will likely add sometime but my plate is full today

@merveenoyan
Copy link
Contributor

@auroraalabs we don't like to put a not written (placeholder) about.md, I think it's not needed because it's already shape of the page and people will not read the page unless it's high quality (it will undermine SEO as well) so no need to put placeholder about.md

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.

3 participants