-
Notifications
You must be signed in to change notification settings - Fork 179
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
in exercises w05, encourage students to try Pos.moved and reflect over results #679
Conversation
Thanks!! I can merge and compile after review (currently you need old latex as installed in 20.04 or later or else allslides.tex fails). I will review this soon. |
This looks great. It would be good if can add some text with explanation in the subtask solution as well? |
Classes don't have methods, only objects and class instances do, strictly speaking... |
Good catch - I was a little uncertain as to what the specific terminology should be, I've changed it in accordance with your suggestion. I've also added an initial draft for an explanation to the solution. Remaining concerns:
|
LGTM |
I will add "[till en referens] till klassen" |
Oh yeah, that makes sense :) |
many thanks for discovering and fixing this! |
This is a first draft to solve #678. Feedback welcome!
I have not yet managed to compile the compendium to ensure that this looks good.