-
Notifications
You must be signed in to change notification settings - Fork 271
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 Spaces to Task Data #561
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, @osanseviero do you have an objection?
@@ -58,6 +58,7 @@ const taskData: TaskDataCustom = { | |||
id: "superb/hubert-large-superb-er", | |||
}, | |||
], | |||
spaces: [], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are the keys for the dict in this list? description & id?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's an ExampleRepo
same as the other repo types
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks! No objections
maybe we could merge after the one on 🌜🛩 is merged 😅 |
Actually this one will need to be merged before https://github.com/huggingface/moon-landing/pull/4861, as the internal one depends on this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i haven't followed too much, the actual data will be filled in some subsequent PRs?
Yes, Merve is curating a list of spaces, which is shared in Slack in the task pages channel |
required for https://github.com/huggingface/moon-landing/pull/4861
This PR add a "spaces" attribute to the TaskData, used for displaying a Spaces section on the Task Page.
Note: I run the linted on the config files, so it may be better to check "Hide whitespace" in the "Files changed" tab :)