search

keras-team / keras-io

Keras documentation, hosted live at keras.io
Apache License 2.0
2.77k stars 2.04k forks source link

Example: video classification - broken link to the dataset #1342

Open mikbuch opened 1 year ago

mikbuch commented 1 year ago

The link to the dataset doesn't work anymore: https://keras.io/examples/vision/video_classification/ (code: https://github.com/keras-team/keras-io/blob/master/examples/vision/video_classification.py).

I contacted the author, we will see what can be done about that.

mikbuch commented 1 year ago

Sayak Paul replied to my e-mail:

Sayak Paul Apr 27, 2023, 4:37 AM (1 day ago) to me

Does this one help?

https://huggingface.co/datasets/sayakpaul/ucf101-subset/resolve/main/UCF101_subset.tar.gz

Sayak Paul | sayak.dev

Mikołaj Buchwald 1:06 PM (0 minutes ago) to Sayak

Thank you for the link! The CSVs with sample names / paths ("train.csv", "test.csv") are missing from the untarred directory, but I keep looking in various links from the dataset page: https://www.crcv.ucf.edu/data/UCF101.php

Best regards, Mikolaj

As I wrote back, I keep looking for the correct link with the full dataset, or with the subset, that will match the example.

mikbuch commented 1 year ago

I can see now also the subsampling notebook (https://colab.research.google.com/github/sayakpaul/Action-Recognition-in-TensorFlow/blob/main/Data_Preparation_UCF101.ipynb). I should be able to run these examples

mikbuch commented 1 year ago

Ok, I was able to run the example.

In order to fix the notebook example in the docs:

  1. Upload the new/proper dataset somewhere, to be able to download it with wget
  2. Create a pull request with updated links, and some additional changes that will make the example more clear, and compatible with newer versions of the modules
mikbuch commented 1 year ago

Ok, I was able to run the example.

In order to fix the notebook example in the docs:

  1. Upload the new/proper dataset somewhere, to be able to download it with wget
  2. Create a pull request with updated links, and some additional changes that will make the example more clear, and compatible with newer versions of the modules
SuryanarayanaY commented 1 year ago

@sayakpaul ,

The link !wget -q https://git.io/JGc31 seems not working as per gist . Could you please check the broken link and update it or create a pull request for updating. Please let me know if any help required from my side.

Thanks!

mikbuch commented 1 year ago

@sayakpaul ,

The link !wget -q https://git.io/JGc31 seems not working as per gist . Could you please check the broken link and update it or create a pull request for updating. Please let me know if any help required from my side.

Thanks!

@SuryanarayanaY , we have a whole PR on that, please revise: https://github.com/keras-team/keras-io/pull/1348 :)