-
Notifications
You must be signed in to change notification settings - Fork 10
Update episode on "Running your first recipe" for compatibility with v2.13.0 #390
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
Conversation
LisaBock
left a comment
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 @rswamina !
I have two small comments but otherwise it looks fine.
| 2025-11-26 09:59:37,413 UTC [3098040] INFO ---------------- | ||
| 2025-11-26 09:59:37,413 UTC [3098040] INFO Reading configuration files from: | ||
| /sw/spack-levante/miniforge3-25.9.1-0-Linux-x86_64-oqcirx/lib/python3.12/site-packages/esmvalcore/config/configurations/defaults (defaults) | ||
| /home/b/b383829/.config/esmvaltool (default user configuration directory) |
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.
Usually we make it anonymously by replacing "b/b383829" with e.g. "user/username".
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.
@LisaBock - I made it anonymous previously because it had an obvious user name. Since the user names on levante do not identify anyone, I left it but can change it.
| ``` | ||
| {% include example_output.txt %} | ||
| ``` | ||
| {: .output} |
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.
Could you collapse the long output?
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.
@LisaBock - I am not able to figure out how to do a drop down and include text from an extrenal file. I tried a few things but couldn't get it to work. Do you know how to do it?
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.
It is not too long and will be useful to see, so if it's ok, we can leave it as is. When we move to .rst format, we can make more changes as needed. Is that ok?
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.
@rswamina It's okay for me. I tried it out, but I also couldn't find a better solution.
You can merge it!
Pull Request checklist
We appreciate your time and effort to improve the tutorial. Please keep in mind that lesson maintainers are volunteers and it may be some time before they can respond to your contribution.
Before you start
Tasks
Lesson developmentsection in CONTRIBUTING.md and does not contain any spelling or grammatical errors.Lesson organizationandLesson formattingsections in CONTRIBUTING.md and does not contain any spelling or grammatical errors.make serve, alternativelymake docker-serve. Please see thePreviewing your changes locallysection in CONTRIBUTING.md for installation instructions.If you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.
Closes [#389]