Skip to content
This repository has been archived by the owner on Mar 24, 2024. It is now read-only.

Remove broken image url #112

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

KevinYang21
Copy link

@KevinYang21 KevinYang21 commented Dec 4, 2023

Remove url pointing to a 404 url

Changes summary

Fixes #ISSUE_NUMBER
#111

Summary of changes in this PR
Remove a broken image url in the list.

The broken image url can lead to errors in the materials. E.g.
Running this cell will get:

UnidentifiedImageError: cannot identify image file '**/ray-educational-materials/Ray_Core/task_images/stennis.jpg'

(image count referred Let's download 100 large images is not updated, as the # of images is 101 even after the removal)

PR scope

  • new notebook
  • improvements to the existing notebook
  • bug fixes
  • other

Checklist

  • notebook is without outputs
  • empty cells are removed
  • code is formatted with black
  • imports are sorted
  • if new notebook is added, README is updated

Remove url pointing to a 404 url

Signed-off-by: Kevin Yang <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant