Skip to content

Conversation

@AndiH
Copy link
Contributor

@AndiH AndiH commented Sep 28, 2020

PGI beyond 20.4 is not called PGI any more but NVIDIA HPC SDK, with 20.7 being the first version; https://developer.nvidia.com/hpc-sdk.

We call the (set of) software(s) NVHPC, after internal discussions and on recommendation by NVHPC's product owners. They call NVIDIA HPC SDK NVHPC internally as well.

This pull request is part of a set of three pull requests to introduce NVHPC to EasyBuild (1, 2, 3). The stack has been successfully deployed at JSC and is under use by our HPC users.
The EasyBuild files are created after discussions with the NVHPC product owners, taking into account HPC practices and EasyBuild conventions.

The toolchain and compiler introduced with this pull request are closely based on PGI, cleaned up for unsupported flags.

@boegel boegel added this to the next release (4.3.1) milestone Sep 29, 2020
@hound hound bot deleted a comment from AndiH Sep 29, 2020
@hound hound bot deleted a comment from boegel Sep 29, 2020
@hound hound bot deleted a comment from boegel Sep 29, 2020
@hound hound bot deleted a comment from AndiH Sep 29, 2020
@AndiH
Copy link
Contributor Author

AndiH commented Sep 29, 2020

I think that should be it.

Tests on our JSC machine with the current state of the files (including renaming "Nvhpc" --> "NVHPC") seemed to work well.

@boegel
Copy link
Member

boegel commented Sep 29, 2020

Hmm, that failing test has nothing to do with the changes made here, I've re-triggered the tests, some kind of race condition?

@boegel boegel changed the title Add NVHPC Infrastructure (aka PGI) add support for NVHPC compiler + toolchain (based on PGI) Sep 30, 2020
@easybuilders easybuilders deleted a comment from boegelbot Oct 12, 2020
@boegel
Copy link
Member

boegel commented Oct 13, 2020

lgtm, thanks @AndiH!

@boegel boegel merged commit f038e8b into easybuilders:develop Oct 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants