release/public-v1: update documentation, update Externals.cfg

ufs-community / ufs-mrweather-app

UFS Medium-Range Weather Application

Other

23 stars 23 forks source link

release/public-v1: update documentation, update Externals.cfg #217

Closed climbfuji closed 4 years ago

climbfuji commented 4 years ago

This PR

updates the documentation w.r.t. setting environment variables on different systems
updates Externals.cfg to point to tags ufs-v1.1.0 for ufs-weather-model, cime, fv3gfs cime interface (with this change, we are using tags ufs-v1.1.0 for all components of the app)

@ligiabernardet I don't think I have my system ready to generate the documentation and check if everything looks ok. Can you check the wording and the layout for me, please?

ligiabernardet commented 4 years ago

@climbfuji Are we proceeding with stopping support for RTs on Stampede? If so, could you update this PR to remove Stampede from the 3 occurrences in Chapter 6 (the chapter about RTs)?

ligiabernardet commented 4 years ago

Documentation changes look good, just waiting on the change related to Stampede for Chapter 6. The formatting is good and can be seen at https://dom-app-ug.readthedocs.io/en/latest/quickstart.html

climbfuji commented 4 years ago

@climbfuji Are we proceeding with stopping support for RTs on Stampede? If so, could you update this PR to remove Stampede from the 3 occurrences in Chapter 6 (the chapter about RTs)?

Now that we most likely solved the RT issue as well, let me go back and try running them before making any changes ...

climbfuji commented 4 years ago

All looks good in this PR and it can be merged now or wait for results of Stampede RT to decide if/what needs to be changed in the documentation.

Throughput on stampede is very slow, hopefully I've got some feedback in the afternoon.

climbfuji commented 4 years ago

The first out of six regression tests passed on Stampede (from part 1). I think we can keep the RT info for Stampede in the documentation and merge as-is.