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

'code' folder should be re-established/replaced #286

Open
tobyhodges opened this issue Jun 14, 2023 · 5 comments
Open

'code' folder should be re-established/replaced #286

tobyhodges opened this issue Jun 14, 2023 · 5 comments

Comments

@tobyhodges
Copy link
Member

@bobturneruk in #221:

some of the code used to generate figures was taken out of the main branch when the repo was upgraded to workbench. It may be worth having a separate issue on this, dealing with where such code should sit in the current repo structure e.g. instructor notes. This would be relevant to any code added resolving this issue not to be run by learners.

@tobyhodges
Copy link
Member Author

I would propose that we recreate a /code directory.

That said, I quite like the idea of including the code for plots as inline Instructor Notes (where that code is not part of the lesson content). If we do take that approach, we should make sure those notes have headings that clearly indicate what they contain.

@bobturneruk
Copy link
Contributor

I like the latter idea. It will encourage us to maintain parity with the figures. I guess details on how to make an inline instructor note are in the workbench docs.

@tobyhodges
Copy link
Member Author

Yes, here: https://carpentries.github.io/sandpaper-docs/episodes.html#instructor-notes

You might also like this helpful video, featuring a lesson repository you may be familiar with 😉

@zkamvar
Copy link
Contributor

zkamvar commented Jun 16, 2023

The code folder was removed in #222, before the transition.

My 2 cents: if the code directory is intended to generate figures for the lesson by maintainers, but not instructors or learners, then it's best to create it at the top level and it will not be included in the rendered lesson site.

if, however you want to include the code as part of the repository, then it should live in the episodes/files folder and you can refer to the file in the episode with a relative link (e.g. [RectMask.py](files/04-drawing/RectMask.py)), which will be valid on The Workbench AND when viewing the markdown preview on GitHub.

@deppen8
Copy link
Contributor

deppen8 commented Aug 7, 2023

I investigated this a bit and it doesn't seem like the code in #222 is the full code for generating all the figures in the lesson. Is it possible that code lived elsewhere or got removed in a different PR?

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

No branches or pull requests

4 participants