Closed garymm closed 4 years ago
Check out this pull request on
Review Jupyter notebook visual diffs & provide feedback on notebooks.
Powered by ReviewNB
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it!
and we'll verify it.
ℹ️ Googlers: Go here for more info.
@googlebot I signed it!
contentsOfCSV -> contentsOf. CSV is already in the function name so this arg name seems needlessly verbose.
Use local variable names in comments about "This should never happen". Saying "we construct " was a bit confusing to me, especially since the param name in this case is also the name of the python module.
Print tensor shapes. I found this helpful to understand the process.