Skip to content

Conversation

dineshladi
Copy link
Contributor

@dineshladi dineshladi commented Apr 30, 2022

  • download parameter is set to True so that data is downloaded
  • transform and target_transform parameters are not supported anymore. Hence, removed from the recipe.
  • root parameter is undefined in the recipe. Hence root parameter value is set to './' so that data is downloaded to the current folder.
  • remove transform parameter related help info in the document

@netlify
Copy link

netlify bot commented Apr 30, 2022

Deploy Preview for pytorch-tutorials-preview ready!

Name Link
🔨 Latest commit 4e7bbd8
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-tutorials-preview/deploys/626d5f26753aab000851246f
😎 Deploy Preview https://deploy-preview-1903--pytorch-tutorials-preview.netlify.app/recipes/recipes/loading_data_recipe
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@msaroufim msaroufim self-requested a review April 30, 2022 17:06
@msaroufim
Copy link
Member

Can you run the new command, show me the pytorch version and paste that it works here? Over time I think we should move to make our tutorials testable @svekars

@dineshladi
Copy link
Contributor Author

dineshladi commented Apr 30, 2022

torch version

import torch
import torchaudio

print(f"torch version: {torch.__version__}")
print(f"torchaudio version: {torchaudio.__version__}")

# torch version: 1.11.0
# torchaudio version: 0.11.0

yesno data download code snippet

torchaudio.datasets.YESNO(
     root='./',
     url='http://www.openslr.org/resources/1/waves_yesno.tar.gz',
     folder_in_archive='waves_yesno',
     download=True)

# <torchaudio.datasets.yesno.YESNO at 0x1100d9dc0>

@msaroufim msaroufim merged commit 6be331b into pytorch:master Apr 30, 2022
@malfet
Copy link
Contributor

malfet commented May 2, 2022

@msaroufim tutorials that end with .py are already testable, so...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants