chore: Update DL notebook#26
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
| @@ -0,0 +1,1978 @@ | |||
| { | |||
There was a problem hiding this comment.
Line #1. covid = sc.read('covid_cite.h5ad')
Put this onto our S3 (all of our datasets should be there) and use the backup URL thingy of read,please.
Reply via ReviewNB
There was a problem hiding this comment.
Sorry, what is a "URL thingy of read"?
There was a problem hiding this comment.
Ah I see. No I just went with a pooch downloader, sorry I assumed this was pointing at gdown.
| @@ -0,0 +1,1978 @@ | |||
| { | |||
There was a problem hiding this comment.
Line #6. for batch in train_loader:
This whole section is very long. Can we reduce it?
Reply via ReviewNB
There was a problem hiding this comment.
I think it is important because it shows how to work with the loader output. Isn't that the point?
| @@ -0,0 +1,1978 @@ | |||
| { | |||
There was a problem hiding this comment.
There was a problem hiding this comment.
I think it's pretty common to print this as a sanity check.
2 participants
That was painful! But the accuracy is better than the last notebook so I think it went alright!