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

Update model card for Cohere #37056

Merged
merged 10 commits into from
Apr 3, 2025
Merged

Conversation

bimal-gajera
Copy link
Contributor

@bimal-gajera bimal-gajera commented Mar 28, 2025

What does this PR do?

This PR updates the model card for Cohere and follows the template outlined in the issue. Please let me know if any changes are required.

As part of this update, I also replaced the outdated blog link https://txt.cohere.com/command-r/ with the updated official blog link https://cohere.com/blog/command-r.

This PR addresses part of issue #36979 by updating the cohere.md model card.

Before submitting

  • This PR improves the docs.

Who can review?

@stevhliu

@github-actions github-actions bot marked this pull request as draft March 28, 2025 00:15
Copy link

Hi 👋, thank you for opening this pull request! The pull request is converted to draft by default. The CI will be paused while the PR is in draft mode. When it is ready for review, please click the Ready for review button (at the bottom of the PR page). This will assign reviewers and trigger CI.

@bimal-gajera
Copy link
Contributor Author

bimal-gajera commented Mar 28, 2025

Note: CI failures are caused by issues unrelated to this PR:

I ran the relevant checks locally:

  • make quality fails due to a docstring mismatch in NougatTokenizerFast, which is not touched in this PR.
  • make repo-consistency fails due to missing optional dependencies in image_processing_utils.

@bimal-gajera bimal-gajera marked this pull request as ready for review March 28, 2025 00:51
@github-actions github-actions bot requested a review from stevhliu March 28, 2025 00:51
Copy link
Member

@stevhliu stevhliu 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 your contribution!

@bimal-gajera
Copy link
Contributor Author

I've made all the suggested changes and resolved the other comments. Let me know if there's anything else you'd like me to update. Thanks again for the feedback!

@bimal-gajera bimal-gajera requested a review from stevhliu April 1, 2025 20:43
@HuggingFaceDocBuilderDev

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.

Copy link
Member

@stevhliu stevhliu left a comment

Choose a reason for hiding this comment

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

Super, just a few more changes!

@stevhliu stevhliu merged commit 2b84831 into huggingface:main Apr 3, 2025
10 checks passed
@bimal-gajera bimal-gajera deleted the cohere-model-card branch April 4, 2025 17:22
yuchenxie4645 pushed a commit to yuchenxie4645/transformers that referenced this pull request Apr 5, 2025
* Update Cohere model card to follow standard template

* Update docs/source/en/model_doc/cohere.md

Co-authored-by: Steven Liu <[email protected]>

* Update docs/source/en/model_doc/cohere.md

Co-authored-by: Steven Liu <[email protected]>

* Update docs/source/en/model_doc/cohere.md

Co-authored-by: Steven Liu <[email protected]>

* Update docs/source/en/model_doc/cohere.md

Co-authored-by: Steven Liu <[email protected]>

* Update docs/source/en/model_doc/cohere.md

Co-authored-by: Steven Liu <[email protected]>

* Update docs/source/en/model_doc/cohere.md

Co-authored-by: Steven Liu <[email protected]>

* Update cohere.md

Update code snippet for AutoModel, quantization, and transformers-cli

* Update cohere.md

* Update docs/source/en/model_doc/cohere.md

Co-authored-by: Steven Liu <[email protected]>

---------

Co-authored-by: Steven Liu <[email protected]>
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