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

Standardize model cards #6898

Closed
wants to merge 2 commits into from

Conversation

bamps53
Copy link
Contributor

@bamps53 bamps53 commented Feb 7, 2024

What does this PR do?

#6891 (WIP)

Before submitting

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@sayakpaul

@bamps53
Copy link
Contributor Author

bamps53 commented Feb 7, 2024

I will create the notebooks after completing all the modifications.

@sayakpaul
Copy link
Member

@bamps53 you don't have work on all at once. You can just pick one at a time and then submit individual PRs. Feel free to comment further on #6891. This is a better approach as it's easier to review the PRs that way.

Also, thanks so much for #6895.

@bamps53
Copy link
Contributor Author

bamps53 commented Feb 8, 2024

Closed as it has to be split for each scripts.

@bamps53 bamps53 closed this Feb 8, 2024
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