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

added streaming example using bokeh server and nyc_taxi.csv #143

Merged
merged 1 commit into from Apr 14, 2016

Conversation

brendancol
Copy link
Collaborator

@brendancol brendancol commented Apr 14, 2016

To run example:

cd examples
bokeh serve stream.py --show

It will take a little bit for the nyc taxi to load into memory. From there, you will need to pan or zoom once to kick off streaming.

screen shot 2016-04-14 at 3 01 31 pm

Things that would make this example better:

  • start streaming immediately instead of need to pan/zoom to start
  • add time slider instead of text and allow user to change

@jbednar
Copy link
Member

jbednar commented Apr 14, 2016

Looks great to me! Nice, simple code, and a cool demo.

Any idea why the data is being clipped across the top of Manhattan in that image? There are data points above that in the dataset...

@brendancol
Copy link
Collaborator Author

brendancol commented Apr 14, 2016

I wonder if I'm working with an older nyc_taxi.csv that got cut off. When I run wc nyc_taxi.csv, I get 10679308 lines in the file. Does that sound correct?

@jbednar
Copy link
Member

jbednar commented Apr 14, 2016

I get 11842095, so yes, probably your nyc_taxi.csv was created using too-tight a bounding box.

@jbednar
Copy link
Member

jbednar commented Apr 14, 2016

That shouldn't affect the code, which I'm happy for you to merge whenever you are ready.

@brendancol brendancol merged commit 55f6282 into holoviz:master Apr 14, 2016
@brendancol brendancol deleted the streaming-example branch April 14, 2016 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants