Skip to content

Conversation

@msimberg
Copy link
Collaborator

@finkandreas do you mind having yourself listed as "code owner" for this page?

@msimberg msimberg requested a review from finkandreas March 21, 2025 17:03
@msimberg msimberg requested review from RMeli and bcumming as code owners March 21, 2025 17:03
@github-actions
Copy link

preview available: https://docs.tds.cscs.ch/56


!!! todo
write some docs
The `linalg` and `linalg-complex` uenvs are similar to the [`prgenv-gnu`][ref-uenv-prgenv-gnu] and [`pregnv-nvfortran`][ref-uenv-prgenv-nvfortran] uenvs in that they don't provide a specific application, but common libraries useful as a base for building other applications.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The `linalg` and `linalg-complex` uenvs are similar to the [`prgenv-gnu`][ref-uenv-prgenv-gnu] and [`pregnv-nvfortran`][ref-uenv-prgenv-nvfortran] uenvs in that they don't provide a specific application, but common libraries useful as a base for building other applications.
The `linalg` and `linalg-complex` uenvs are similar to the [`prgenv-gnu`][ref-uenv-prgenv-gnu] uenvs in that they don't provide a specific application, but common libraries useful as a base for building other applications.

I know what you are trying to say, but just to avoid possible question on where is nvhpc in these packages etc.?

Copy link
Member

Choose a reason for hiding this comment

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

I don't know if you can avoid those questions. Providing a link to the nvfortran uenv might even answer the "where can I find nvhpc?" question.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't feel very strongly, but I think @bcumming has at least done a good job on the prgenv-gnu and prgenv-nvfortran pages saying you probably want prgenv-gnu at the top of the pages.

I've at least fixed the pregnv-nvfortran typo here for now. Let me know if you still think it's worth removing the nvfortran mention.

@github-actions
Copy link

preview available: https://docs.tds.cscs.ch/56

docs/software/communication @msimberg
docs/services/firecrest @jpdorsch @ekouts
docs/software/communication @msimberg
docs/software/prgenv/linalg.md @finkandreas @msimberg
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@finkandreas note that I added you here tentatively.

@github-actions
Copy link

preview available: https://docs.tds.cscs.ch/56

@bcumming bcumming merged commit 5f9fa2a into main Mar 24, 2025
1 check passed
@bcumming bcumming deleted the linalg branch March 24, 2025 10:16
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