alan-turing-institute / rds-course

Materials for Turing's Research Data Science course
https://alan-turing-institute.github.io/rds-course/
31 stars 13 forks source link

Module 3 #50

Closed crangelsmith closed 3 years ago

crangelsmith commented 3 years ago

Fixes issue #19

Content

Proofread and ready for external review

review-notebook-app[bot] commented 3 years ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

crangelsmith commented 3 years ago

I would like to add this figure to the context discussion of 3.4. Currently, we have only one figure talking about context but I feel is not representative of all these issues.

The problem with this figure is that shows a sharp trend of something that is very small, people can overreact if they don't compare it with the number of universities and invited speakers there are usually on a single year.

jack89roberts commented 3 years ago

Hi here, I was trying to build the book from this branch and got an error due to not having the clase25datos kernel used in 3.5-DataVisForExploration.ipynb, could it be set to a default Python 3 (ipykernel) like the others?

crangelsmith commented 3 years ago

Hi @jack89roberts, yes that is my fault, I was piggybacking from one of my existing kernels but shouldn't have committed it. We should definitely move it to the default.