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

Edited UM content in atmosphere.md #454

Merged
merged 3 commits into from
Jul 20, 2023
Merged

Conversation

heidinett
Copy link
Contributor

Edited UM content in atmosphere.md

Fixes Issue #344

@heidinett heidinett requested a review from MartinDix July 19, 2023 22:34
@github-actions
Copy link

github-actions bot commented Jul 19, 2023

PR Preview Action v2.1.1
🛬 Preview removed because the pull request was closed.
2023-07-20 01:51 UTC

@MartinDix
Copy link
Contributor

Definitely better

Delete climate modelling at end of first sentence because equally applicable to global NWP.

Perhaps delete subgrid-scale in the second sentence (a bit technical and not quite true for radiation)

The Australian Bureau of Meteorology (BoM) uses the UM in its suite of ACCESS forecast systems for daily and seasonal ...
Add a link to http://www.bom.gov.au/australia/charts/viewer/

I think the STASH register link is too technical here.

@heidinett
Copy link
Contributor Author

@MartinDix I have made your suggested changes. Could you please check content again before merging.

@atteggiani atteggiani merged commit cf81b70 into development Jul 20, 2023
2 checks passed
@atteggiani
Copy link
Contributor

Merging this already.
@MartinDix if you have any more comments, please leave them in the new slack channel.

@atteggiani atteggiani deleted the dev/heidi/issue344 branch September 27, 2023 05:48
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