Skip to content
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

Tutorial feedback response to #1219 #1410

Merged
merged 2 commits into from
May 3, 2021

Conversation

m4reko
Copy link

@m4reko m4reko commented Apr 28, 2021

Feedback response

Authors: Markus Wesslén (mwesslen@kth.se), Lara Rostami (larasm@kth.se, @LaraRos)

We received feedback on our tutorial (proposal: #1015, submission: #1213) from August and Christian which can be seen in #1219

We modified the following after the feedback:

  • Clarified introduction
  • Added setup script to get rid of some deprecation errors
  • Encouraged the user to test more parts of the driver themselves
  • Fixed a few typos
  • Clarified why the test pages used are good for testing
  • Clarified the importance of copying the code snippets to the correct location
  • Added more resources for continued exploration of the subject to the finishing text

The changes can be seen in this commit

@m4reko
Copy link
Author

m4reko commented Apr 30, 2021

We are ready for grading @Deee92

@khaes-kth khaes-kth self-assigned this May 3, 2021
@khaes-kth khaes-kth merged commit a891c84 into KTH:2021 May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants