-
Notifications
You must be signed in to change notification settings - Fork 4
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
Changes from dry run #15
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @MarionBWeinzierl a comment in there.
Also, I forget the exact discussion, but on reflection here I would definitely cut the 'config file' part as this is just TMI at the end of a packed session and never lands very well.
Leaving the slide in to be visible is an option, but I am certain I would not cover it.
If anything I think fstrings would be easier for people to grasp if we somehow find ourselves with extra time.
Any preferences about leaving the slides in and skipping them, or commenting them out for the case you have time? |
For the time being I have removed both. However, not sure whether the title of the section is then still fitting (maybe with the "honorable mentions" it is), or whether it should be changed. Or another slide added to mention the stuff that we removed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Punch it Chewie.
@MarionBWeinzierl I can't see we will have lots of spare time, so both removed is probably fine. We can always mention or demo if we feel the need to.
At this point let's get something deliverable for the summer school. We can refine and add back at a later date after next week if so desired.
Closes #14 . Implement some of the changes from the dry run (see issue)