-
Notifications
You must be signed in to change notification settings - Fork 13
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
[WIP] Streamlining #138
[WIP] Streamlining #138
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Accidental dev env file
Still think 1.1 has too much, but not sure! Need to think about how this is taught and how much material is needed. |
@callummole I just realised I've been merging back-to-back markdown cells - I'm guessing this will make it super annoying with respect to #126 for making slides? I'll stop now, but if we do come up with suggestions or guidelines for teaching / prepping slides do let me know, and I'll adjust accordingly (only did this to 1.1. so far, and a few times in other places) |
As per meeting this morning with @callummole - he suggested we use a glossary to store a lot of the removed information caused by streamlinning the notebooks! |
quick thought c503cfa and some others have changed names. Can we make a note here to double-check there's no hard references in the book which reference to no dead files?
|
I noticed that some grammar things I'd previous fixed had become undone - partly fixed by 044550f |
Latest commit: 7c37bc3 still has Will merge this branch once all files are reviewed and verified |
This is an ambitious, and
possiblystupid suggestion.Thus, I'd suggest each changed file be thoroughly reviewed.
(Which I will do! Would be rude to ask anyone else to check I haven't broken everything.)
m1
m2
m3
m4