ADI-Labs / density

wireless density API
13 stars 26 forks source link

Blog Post - Ipython Notebook #132

Closed jzf2101 closed 9 years ago

jzf2101 commented 9 years ago

Closes #112 See discussion and notebook.

Notebook link here to preview

natebrennand commented 9 years ago

are we just linking to the iPython notebook or making an HTML page hosting this?

thebrianzeng commented 9 years ago

we can put the markdown as a blog post on adicu.com/blog and then link to both in the advertising.

On Sat, Mar 21, 2015 at 2:36 PM jzf2101 notifications@github.com wrote:

Closes #112 https://github.com/adicu/density/issues/112 See discussion and notebook.

Notebook link here http://nbviewer.ipython.org/github/jzf2101/density/blob/master/datasci/density_demo.ipynb

to preview

You can view, comment on, or merge this pull request online at:

https://github.com/adicu/density/pull/132 Commit Summary

  • adding the ipython notebook of i was going to share to devfest
  • updated language to clarify average space plot re nates comments
  • changed calculation of total plot across columbia to be average number of devices at columbia
  • fixed title of plot for devices across columbia
  • changed language regarding buildings in demo
  • pulling upstream from nate's commits Merge remote-tracking branch 'upstream/master'
  • modifying to add Brians comments
  • added markdown version of blogpost with attached images
  • added updates to ipython notebook version of blogpost
  • merging changes from map commits Merge remote-tracking branch 'upstream/master'
  • merging in updates
  • made additional edits to density blogpost and demo to add chris mulligans demo
  • updated blog link to notebook

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/adicu/density/pull/132.

jzf2101 commented 9 years ago

@thebrianzeng I think using the markdown file as a blog post is great idea. Could someone show me how we post to the blog? It'd take extra work for me to figure out how to host the notebook on our own server, but we could do that if we want to.

jzf2101 commented 9 years ago

133 seems to be related to the changes I made in this pull request, so I think I'll need to rework it so it can still run. I repulled master and was able to run.

natebrennand commented 9 years ago

There are no changes to any of the python files in the PR. Are there changes that you have not pushed to this branch?

jzf2101 commented 9 years ago

nvm that has nothing to do with it.

jzf2101 commented 9 years ago

@thebrianzeng Updated the language with your suggestions. Could you look again and confirm it's cool with you?

evantarrh commented 9 years ago

:+1: