Replace md ipynb - Githubissues

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

Replace md ipynb #123

Closed AoifeHughes closed 2 years ago

AoifeHughes commented 2 years ago

This replaced all the md files in m2 with ipynb native files.

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

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

AoifeHughes commented 2 years ago

[x] I still need to proof this and check everything hasn't broken

AoifeHughes commented 2 years ago

I think this is fine, it's quite a substantial change though. Could someone else handle this in case I'm way off?

callummole commented 2 years ago

Great! Do you mind making everything as a notebook? (i.e. even the markdown files that do not have code). This will make it easier to make a slides companion to each module.

AoifeHughes commented 2 years ago

✨ Fantastic idea ✨ On it now!

AoifeHughes commented 2 years ago

@callummole Commit https://github.com/alan-turing-institute/rds-course/pull/123/commits/4ec5c2b37d65ee03bfd8a629e8198c4210cfe645 does this.

AoifeHughes commented 2 years ago

probably

callummole commented 2 years ago

On a local build and (cursory) look through this all seems great.