Skip to content

Conversation

MrShahzebKhoso
Copy link
Contributor

  • Added about.md with a definition and summary of the task.
  • Added data.ts with task metadata:
    • Datasets (VIRESET, LongV-EVAL, SeedVR_VideoDemos)
    • Models (e.g., Lucy-Edit-Dev, SeedVR2, Shape-for-Motion, etc.)
    • Demo Spaces (lucy-edit-dev, SeedVR2-3B)
    • Task summary describing scope and applications.

@pcuenca
Copy link
Member

pcuenca commented Oct 1, 2025

cc @merveenoyan @sergiopaniego

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.

very neat and thanks a ton for opening this PR!
these pages are targeted for devs to get started from scratch so let's explain a bit better and from scratch!

Copy link
Member

@sergiopaniego sergiopaniego left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! I agree with Merve's suggestions and just leaving a few additional minor nits :)

@MrShahzebKhoso
Copy link
Contributor Author

Thank you for considering this PR and taking time to review.

Updated both the about.md and data.ts files based on all the helpful feedback from @merveenoyan and @sergiopaniego — thank you both for the detailed and thoughtful reviews. Really appreciate the guidance.

Also added a Video Super-Resolution demo (low-res to high-res) via PR #10
on the datasets repo.

Everything is now cleaned up, clarified, and aligned with the review suggestions.

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.

make sure that the changes reflect here: as well packages/tasks/src/tasks/index.ts

cc: @merveenoyan

@MrShahzebKhoso
Copy link
Contributor Author

Thanks for catching that, @Vaibhavs10!
I’ve added the corresponding entry to packages/tasks/src/tasks/index.ts so the changes are now reflected there as well.
Appreciate the review and guidance!

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.

thank you for re-iterating!

@MrShahzebKhoso
Copy link
Contributor Author

Thank you, @merveenoyan for the review. I have made the changes including:

  • Linking the Lucy-Edit-Dev, and providing the context on what it's trained on.
  • Removing the second inference example and adding the sentence to direct to the model cards and try example codes provided.
  • Removing '##Repositories' and adding a 'read more' sentence.
  • Removing older model from model.ts.
    With all these changes, I believe the PR is now ready to be merged. If any further feedback or adjustments are needed, I’d be happy to iterate!

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