Skip to content

update model card for gpt-j - #40404

Closed
nayana1729 wants to merge 4 commits into
huggingface:mainfrom
nayana1729:documentation-update-gptj
Closed

nayana1729 wants to merge 4 commits into
huggingface:mainfrom
nayana1729:documentation-update-gptj

Conversation

@nayana1729

Copy link
Copy Markdown
Contributor

What does this PR do?

This PR updates the documentation for the model GPT-J according to #36979 to improve the model cards with a consistent format.

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).

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.

@stevhliu

@stevhliu stevhliu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for your contribution!

Comment thread docs/source/en/model_doc/gptj.md Resolved
Comment thread docs/source/en/model_doc/gptj.md Resolved
Comment thread docs/source/en/model_doc/gptj.md Resolved
Comment thread docs/source/en/model_doc/gptj.md Resolved
Comment thread docs/source/en/model_doc/gptj.md Resolved
Comment thread docs/source/en/model_doc/gptj.md Resolved
Comment thread docs/source/en/model_doc/gptj.md Resolved
Comment thread docs/source/en/model_doc/gptj.md Resolved
@nayana1729

Copy link
Copy Markdown
Contributor Author

Thank you for the suggestions! Just incorporated them. Let me know if there are any changes to be made.

@stevhliu stevhliu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, just a few more minor changes and we can merge!

Comment thread docs/source/en/model_doc/gptj.md Resolved
Comment thread docs/source/en/model_doc/gptj.md Resolved
@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@ArthurZucker

Copy link
Copy Markdown
Collaborator

Thanks for the PR, and sorry it didn't make it in! This branch is based on v4, and v5 is very different, so rather than rebasing we'd love to invite you to open a new PR against main using our latest tools and good practices:

Closing this one in the meantime, thanks again!

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.

4 participants