Skip to content
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

Chapter 04 - Building a CNN-based architecture using PyTorch #57

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cpoles
Copy link

@cpoles cpoles commented Aug 29, 2022

Changed variable name within train_batch() function body to match the argument in the function signature.

cpoles added 2 commits August 29, 2022 15:34
Changed variable name within function body to match the argument in the function signature
Prediction tensor within train_batch function call needs to have its first dimension squeezed before passing it to the loss function
@cpoles cpoles changed the title Variable name changed Chapter 04 - Building a CNN-based architecture using PyTorch Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant