Skip to content

Conversation

@davidhassell
Copy link
Contributor

Model documentation for #20

@davidhassell
Copy link
Contributor Author

Hi @charliepascoe - couldn't add you as a reviewer, for some reason, so pinging here!

@charliepascoe
Copy link

It isn't clear to me what I am being asked to do.

@matthew-mizielinski
Copy link
Collaborator

Hi @charliepascoe - couldn't add you as a reviewer, for some reason, so pinging here!

I've just invited @charliepascoe to this repo -- apologies for the oversight.

@davidhassell
Copy link
Contributor Author

It isn't clear to me what I am being asked to do.

Sorry - my fault! It'd be great if you could read the short PR (EMD description for the CMIP7 guidance web pages) and see if it makes sense :)

Copy link
Collaborator

@matthew-mizielinski matthew-mizielinski left a comment

Choose a reason for hiding this comment

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

A few tweaks that occurred to me

@JamesAnstey JamesAnstey changed the title Guidance for modellers: Essential Model Documnentation (EMD) Guidance for modellers: Essential Model Documentation (EMD) Oct 23, 2025
@charliepascoe
Copy link

charliepascoe commented Oct 29, 2025

I have time for doing the review of this now. How do I see the latest version?
I worked it out...
Towards the end of line 108 I would add EMD to the the phrase "full specification" so that it reads "full EMD specification".

@matthew-mizielinski
Copy link
Collaborator

@davidhassell, If you are able to pick up @charliepascoe's last suggestion and are happy with the content I'll resolve conflicts (heading number changes) and get this merged and deployed.

@matthew-mizielinski matthew-mizielinski merged commit 5baff08 into WCRP-CMIP:main Nov 4, 2025
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.

4 participants