Skip to content

remove SLES12 from readme#797

Closed
weiyuan-jiang wants to merge 1 commit intodevelopfrom
feature/wjiang/rm_sles12
Closed

remove SLES12 from readme#797
weiyuan-jiang wants to merge 1 commit intodevelopfrom
feature/wjiang/rm_sles12

Conversation

@weiyuan-jiang
Copy link
Contributor

No description provided.

@weiyuan-jiang weiyuan-jiang requested a review from a team as a code owner March 11, 2025 12:36
Copy link
Contributor

@gmao-rreichle gmao-rreichle left a comment

Choose a reason for hiding this comment

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

Thanks, @weiyuan-jiang, for putting this together. I added a couple of inline comments/question.

```
cd ./GEOSldas
parallel_build.csh
./parallel_build.csh -mil
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess initially we will keep using -mil, but looking ahead, do we really need to have the -mil flag for parallel_build.csh if there's only SLES15?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we will see how build.csh handle the default. @mathomp4

This checks out all the external repositories of the model (albeit only on the first run, [see subsection on mepo below](#mepo)!) and then builds and installs the model.

At **NCCS**, the default is to build GEOSldas on SLES12 (Skylake or Cascade Lake nodes); to build GEOSldas on SLES15 (Milan nodes), use `parallel_build.csh -mil`.
The resulting model build is found in `build-SLES15/`, and the installation is found in `install-SLES15/`, with setup scripts like `ldas_setup` in `install-SLES15/bin`.
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm a bit confused by the build and install directory names, which may be outdated. When I look at the build of the LDAS nightly tests, there's not "-SLES15" in the directory names:

discover36:~ ll /discover/nobackup/mathomp4/SystemTests/builds/LDAS_CONUS/CURRENT/GEOSldas/ |grep build
drwxr-xr-x 13 mathomp4 g0620    4096 Mar 10 23:07 build-Aggressive/
drwxr-xr-x 13 mathomp4 g0620    4096 Mar 10 22:40 build-Debug/
drwxr-xr-x 13 mathomp4 g0620    4096 Mar 10 20:23 build-Release/

The resulting model build is found in `build[-SLESxx]/`, and the installation is found in `install[-SLESxx]/`, with setup scripts like `ldas_setup` in `install[-SLESxx]/bin`.

To obtain a build that is suitable for debugging, use `parallel_build.csh -debug`, which builds in `build-Debug[-SLESxx]/` and installs in `install-Debug[-SLESxx]/`. There is also an option for aggressive optimization. For details, see the [GEOSldas Wiki](https://github.com/GEOS-ESM/GEOSldas/wiki).
To obtain a build that is suitable for debugging, use `./parallel_build.csh -debug -mil`, which builds in `build-Debug-SLES15/` and installs in `install-Debug-SLES15/`. There is also an option for aggressive optimization. For details, see the [GEOSldas Wiki](https://github.com/GEOS-ESM/GEOSldas/wiki).
Copy link
Contributor

Choose a reason for hiding this comment

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

see comments above re. the -mil flag and the -SLES15 string in the dir names



a) At **NCCS**, GEOSldas must be built, configured, and run on the same operating system. To run GEOSldas on Milan nodes (SLES15), start with `ssh discover-mil`.
a) To run GEOSldas on Milan nodes (SLES15), start with `ssh discover-mil`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need the -mil flag? Presumably, when Discover comes back on Thursday, ssh discover should be fine


```
cd (build_path)/GEOSldas/install[-SLESxx]/bin
cd (build_path)/GEOSldas/install-SLES15/bin
Copy link
Contributor

Choose a reason for hiding this comment

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

see comment above re. -SLES15

@weiyuan-jiang
Copy link
Contributor Author

It is merged into the PR #796

@gmao-rreichle gmao-rreichle deleted the feature/wjiang/rm_sles12 branch June 17, 2025 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants