Skip to content

Update README_hd.md#26872

Merged
stevhliu merged 2 commits intohuggingface:mainfrom
biswabaibhab007:patch-1
Oct 20, 2023
Merged

Update README_hd.md#26872
stevhliu merged 2 commits intohuggingface:mainfrom
biswabaibhab007:patch-1

Conversation

@biswabaibhab007
Copy link
Contributor

  • Fixed broken links.

I hope this small contribution adds value to this project.

- Fixed broken links
I hope this small contribution adds value to this project.
@ArthurZucker ArthurZucker requested a review from stevhliu October 18, 2023 09:15
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 fixing the links!

One more minor change and then we can merge :)

Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
@biswabaibhab007
Copy link
Contributor Author

I've committed the suggestion you can merge it now.

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.

Awesome thanks!

@biswabaibhab007
Copy link
Contributor Author

@stevhliu btw on line 90 there's jax installation if that's what you meant you .

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.

We should try to keep the translation as close to the English one as possible. There's a mention of Jax installation on line 259 🙂

I added a link to the Jax doc for you if you don't mind, let me know if that translates correctly!

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 again for the fix!

@stevhliu stevhliu merged commit 224794b into huggingface:main Oct 20, 2023
EduardoPach pushed a commit to EduardoPach/transformers that referenced this pull request Nov 19, 2023
* Update README_hd.md

- Fixed broken links
I hope this small contribution adds value to this project.

* Update README_hd.md

Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>

---------

Co-authored-by: Steven Liu <59462357+stevhliu@users.noreply.github.com>
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