Skip to content

Update episode 9 "CMORization" for v2.14.0 - #420

Merged
LisaBock merged 4 commits into
mainfrom
update_epsiode_09
Apr 14, 2026
Merged

Update episode 9 "CMORization" for v2.14.0#420
LisaBock merged 4 commits into
mainfrom
update_epsiode_09

Conversation

@LisaBock

@LisaBock LisaBock commented Apr 10, 2026

Copy link
Copy Markdown
Member

Pull Request checklist


Before you start

  • Read CONTRIBUTING.md.
  • Create an issue 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 #415

@LisaBock
LisaBock requested review from hb326 and rswamina April 10, 2026 11:11
Comment thread _episodes/09-cmorization.md Outdated
data:
dkrz:
type: esmvalcore.io.local.LocalDataSource
rootpath: ~/data/OBS

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this not be "rootpath: ~/data/OBS6"?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At this point OBS6 and OBS data is in the same directory as we have it on Levante as well.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, good to know!

Comment thread _episodes/09-cmorization.md Outdated

@hb326 hb326 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most of it looks ok, I would say. There are two instances where I am not sure if there are typos or if the specified paths really should be as listed...

@LisaBock
LisaBock requested a review from hb326 April 14, 2026 09:02

@hb326 hb326 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fine with me now. Thanks, @LisaBock!

@LisaBock
LisaBock merged commit f2355da into main Apr 14, 2026
1 check failed
@LisaBock
LisaBock deleted the update_epsiode_09 branch April 14, 2026 12:33
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.

Update "CMORization" epsiode for compatibility with ESMValTool v2.14.0

2 participants