Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions guides/ipynb/keras_hub/hugging_face_keras_integration.ipynb
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,21 @@
"**Description:** How to load and run inference from KerasHub model checkpoints hosted on the HuggingFace Hub."
]
},
{
"cell_type": "markdown",
"metadata": {
"colab_type": "text"
},
"source": [
"## Video Guide\n",
"\n",
"[https://www.youtube.com/watch?v=OpSddL3Lv7I](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)\n",
"\n",
"[![How to use KerasHub with Hugging Face](http://img.youtube.com/vi/OpSddL3Lv7I/maxresdefault.jpg)](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)\n",
Comment on lines +25 to +27
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

To improve clarity and security, the redundant plain text link can be removed, and the image URL should be updated to use https.

Specifically:

  1. The text link on line 25 is redundant as the image below also links to the video.
  2. The image URL on line 27 uses http. It's better to use https to prevent mixed content issues.

"\n",
"\n"
]
},
{
"cell_type": "markdown",
"metadata": {
Expand Down
7 changes: 7 additions & 0 deletions guides/keras_hub/hugging_face_keras_integration.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,13 @@
"""

"""

## Video Guide

[https://www.youtube.com/watch?v=OpSddL3Lv7I](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)

[![How to use KerasHub with Hugging Face](http://img.youtube.com/vi/OpSddL3Lv7I/maxresdefault.jpg)](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)
Comment on lines +14 to +16
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

To improve clarity, the redundant plain text link can be removed. Additionally, the image URL should be updated to use https for security.

Suggested change
[https://www.youtube.com/watch?v=OpSddL3Lv7I](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)
[![How to use KerasHub with Hugging Face](http://img.youtube.com/vi/OpSddL3Lv7I/maxresdefault.jpg)](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)
[![How to use KerasHub with Hugging Face](https://img.youtube.com/vi/OpSddL3Lv7I/maxresdefault.jpg)](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)


## Introduction

KerasHub has built-in converters for HuggingFace's `.safetensors` models.
Expand Down
6 changes: 6 additions & 0 deletions guides/md/keras_hub/hugging_face_keras_integration.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,13 @@

<img class="k-inline-icon" src="https://colab.research.google.com/img/colab_favicon.ico"/> [**View in Colab**](https://colab.research.google.com/github/keras-team/keras-io/blob/master/guides/ipynb/keras_hub/hugging_face_keras_integration.ipynb) <span class="k-dot">•</span><img class="k-inline-icon" src="https://github.com/favicon.ico"/> [**GitHub source**](https://github.com/keras-team/keras-io/blob/master/guides/keras_hub/hugging_face_keras_integration.py)

---

## Video Guide

[https://www.youtube.com/watch?v=OpSddL3Lv7I](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)

[![How to use KerasHub with Hugging Face](http://img.youtube.com/vi/OpSddL3Lv7I/maxresdefault.jpg)](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)
Comment on lines +15 to +17
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

To improve clarity, the redundant plain text link can be removed. Additionally, the image URL should be updated to use https for security.

Suggested change
[https://www.youtube.com/watch?v=OpSddL3Lv7I](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)
[![How to use KerasHub with Hugging Face](http://img.youtube.com/vi/OpSddL3Lv7I/maxresdefault.jpg)](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)
[![How to use KerasHub with Hugging Face](https://img.youtube.com/vi/OpSddL3Lv7I/maxresdefault.jpg)](https://www.youtube.com/watch?v=OpSddL3Lv7I&list=PL9f9vdy2ar1IdH6O6kQGn4Ln-_oLH_Ra3&index=2)


---
## Introduction
Expand Down