-
Notifications
You must be signed in to change notification settings - Fork 4
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
clarify cell 1 #42
Comments
i think it might be instructive if it were broken up into 2 cells:
to give people a sense of how long it takes to do each of them? |
Download takes only 10 seconds or so, most of the time is the processing. It's currently in 1 cell because it is literally running the demo script which pulls data and computes it (because in practice people wanting the demo want to type 1 line rather than 2). In my opinion it doesn't make sense to separate, but what do you think/want? I can make anything happen. :) |
I see.
Maybe just say download takes 10 sec to clarify?
Maybe link to pipeline schematic , so people know the steps that are
happening?
…On Sun, Dec 11, 2016 at 3:07 PM Greg Kiar ***@***.***> wrote:
Download takes only 10 seconds or so, most of the time is the processing.
It's currently in 1 cell because it is literally running the demo script
which pulls data and computes it (because in practice people wanting the
demo want to type 1 line rather than
2).
In my opinion it doesn't make sense to separate, but what do you
think/want? I can make
*anything* happen. :)
—
You are receiving this because you authored the thread.
Reply to this email directly,
view it on GitHub
<#42 (comment)>, or
mute the thread
<https://github.com/notifications/unsubscribe-auth/AACjctDl-NvYV38BMHHkyOCmdkACG0jvks5rHFewgaJpZM4LJ19x>
.
|
what is happening that is taking 3-4 minutes?
seems like most of it is downloading?
does it make sense to break it up into multiple cells?
The text was updated successfully, but these errors were encountered: