Skip to content

Conversation

@rswamina
Copy link
Contributor

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

  • Read CONTRIBUTING.md.
  • Create an #389 to discuss your idea. This allows your contributions to be incorporated into the tutorial.

Tasks

  • Give this pull request a descriptive title.
  • If you are contributing to existing lesson materials, please make sure the content conforms to the Lesson development section in CONTRIBUTING.md and does not contain any spelling or grammatical errors.
  • If you are making a new episode, please make sure the content conforms to the Lesson organization and Lesson formatting sections in CONTRIBUTING.md and does not contain any spelling or grammatical errors.
  • Preferably Codacy checks pass. Status can be seen below your pull request. If there is an error, click the link to find out why.
  • Preview changes on your machine before pushing them to GitHub by running make serve, alternatively make docker-serve. Please see the Previewing your changes locally section in CONTRIBUTING.md for installation instructions.
  • All code instructions have been tested.

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]

@rswamina rswamina self-assigned this Nov 26, 2025
@rswamina rswamina marked this pull request as ready for review November 26, 2025 15:39
@rswamina rswamina requested review from LisaBock and hb326 November 26, 2025 15:39
Copy link
Member

@LisaBock LisaBock left a 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)
Copy link
Member

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".

Copy link
Contributor Author

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.

Comment on lines +53 to +58
```
{% include example_output.txt %}
```
{: .output}
Copy link
Member

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?

Copy link
Contributor Author

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?

Copy link
Contributor Author

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?

Copy link
Member

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!

@rswamina rswamina marked this pull request as draft November 27, 2025 10:13
@rswamina rswamina marked this pull request as ready for review November 27, 2025 11:28
@rswamina rswamina merged commit fa6d73a into main Nov 27, 2025
1 check passed
@rswamina rswamina deleted the update_running_first_recipe_v2.13.0 branch November 27, 2025 11:29
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.

3 participants