Skip to content

Test the Loading data in PyTorch tutorial #2905

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
sekyondaMeta opened this issue Jun 4, 2024 · 3 comments · Fixed by #2922
Closed

Test the Loading data in PyTorch tutorial #2905

sekyondaMeta opened this issue Jun 4, 2024 · 3 comments · Fixed by #2922

Comments

@sekyondaMeta
Copy link
Contributor

sekyondaMeta commented Jun 4, 2024

Test the following tutorial:
https://pytorch.org/tutorials/recipes/recipes/loading_data_recipe.html
Follow these steps:

  1. Fork the pytorch/tutorials repo.
  2. Switch to a new branch.
  3. Run the tutorial as a python script:
python3 <path-to-tutorial>
  1. Save the output into a .txt file. Did you see any errors? Add them to the output.
  2. Run the tutorial in the Google Colab and save the outputs into a .txt file. Did you see any issues? Take screenshots of the issues.
  3. Did the tutorial make sense? Can you suggest any improvements? Past all the outputs, screenshots, and suggestions for improvements in the issue and ping one of the reviewers. If your suggestion is approved, please submit a PR with the fix.

cc @svekars @kit1980

@NeoKish
Copy link
Contributor

NeoKish commented Jun 4, 2024

/assigntome

@svekars
Copy link
Contributor

svekars commented Jun 7, 2024

This recipe specifically is quite outdated and likely does not answer what a user is trying to learn. There is a better tutorial on this topic: https://pytorch.org/tutorials/beginner/basics/data_tutorial.html - we could consider deprecating and creating a redirect from this tutorial to the data_tutorial.html.

@loganthomas
Copy link
Contributor

/assigntome

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

Successfully merging a pull request may close this issue.

4 participants