Closed svenvanderburg closed 11 months ago
Thank you for your pull request :smiley:
:robot: This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.
If you have files that automatically render output (e.g. R Markdown), then you should check for the following:
:mag: Inspect the changes: https://github.com/carpentries-incubator/deep-learning-intro/compare/md-outputs..md-outputs-PR-376
The following changes were observed in the rendered markdown documents:
5-outlook.md (new) | 97 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
md5sum.txt | 3 +-
2 files changed, 99 insertions(+), 1 deletion(-)
:stopwatch: Updated at 2023-08-16 12:17:40 +0000
Is course a better word than lesson? Lesson to me is more indicative of a part of the teaching i.e. an episode while course is broader.
Is course a better word than lesson? Lesson to me is more indicative of a part of the teaching i.e. an episode while course is broader.
Yeah, this is carpentries naming. This whole repo is called one 'lesson', a single chapter is called an 'episode'. But if you are confused than learners are also confused, so I will change it to 'course'.
@cpranav93 is this PR OK apart from the naming of 'lesson' vs 'course'?
Hey Sven, its all good. One point would be to add some insights/answer to the optional question in the challenge but I will create a separate issue for that as it is not mandatory. Merging this now. Thanks for all the hardwork on this! It really looks great.
Fixes #48 and #362