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

Most LUA-format modulefiles have incorrect variables (compiler and MPI). #413

Closed
aerorahul opened this issue Jun 17, 2022 · 2 comments · Fixed by #414
Closed

Most LUA-format modulefiles have incorrect variables (compiler and MPI). #413

aerorahul opened this issue Jun 17, 2022 · 2 comments · Fixed by #414
Assignees
Labels
bug Something isn't working

Comments

@aerorahul
Copy link
Contributor

aerorahul commented Jun 17, 2022

load(pathJoin("hpc-gnu", hpc_gnu_ver))

load(pathJoin("hpc-mpich", hpc_mpich_ver))

Variables hpc_gnu_ver and hpc_mpich_ver are not defined.

local hpc_intel_ver=os.getenv("hpc_gnu_ver") or "9.2.0"

local impi_ver=os.getenv("hpc_mpich_ver") or "3.3.2"

should be updated to be:

local hpc_gnu_ver=os.getenv("hpc_gnu_ver") or "9.2.0"
local hpc_mpich_ver=os.getenv("hpc_mpich_ver") or "3.3.2"
@aerorahul aerorahul changed the title GNU modulefile for hera does not load MPI GNU modulefile for hera has incorrect variables. Jun 17, 2022
@aerorahul aerorahul added the bug Something isn't working label Jun 17, 2022
@MichaelLueken MichaelLueken changed the title GNU modulefile for hera has incorrect variables. Most LUA-format modulefiles have incorrect variables (compiler and MPI). Jun 17, 2022
@MichaelLueken
Copy link
Contributor

@aerorahul - Looking more carefully, this is an issue with most of the LUA-format modulefiles, especially with compiler and MPI versions. I will be addressing all incorrect variables in this issue.

MichaelLueken added a commit to MichaelLueken/GSI that referenced this issue Jun 17, 2022
@MichaelLueken
Copy link
Contributor

In 67b6146:

  • gsi_hera.gnu.lua - replaced:
local hpc_intel_ver=os.getenv("hpc_gnu_ver") or "9.2.0"
local impi_ver=os.getenv("hpc_mpich_ver") or "3.3.2"

with:

local hpc_gnu_ver=os.getenv("hpc_gnu_ver") or "9.2.0"
local hpc_mpich_ver=os.getenv("hpc_mpich_ver") or "3.3.2"
  • gsi_hera.intel.lua - replaced:

local impi_ver=os.getenv("hpc_impi_ver") or "2018.0.4"
with:
local hpc_impi_ver=os.getenv("hpc_impi_ver") or "2018.0.4"

  • gsi_orion.lua - replaced:

local impi_ver=os.getenv("hpc_impi_ver") or "2018.4"
with:
local hpc_impi_ver=os.getenv("hpc_impi_ver") or "2018.4"

  • gsi_s4.lua - replaced:

local impi_ver=os.getenv("hpc_impi_ver") or "18.0.4"
with:
local hpc_impi_ver=os.getenv("hpc_impi_ver") or "18.0.4"

  • gsi_wcoss_dell_p3.lua - replaced:
local hpc_intel_ver=os.getenv("hpc_ips_ver") or "18.0.1.163"
local impi_ver=os.getenv("hpc_impi_ver") or "18.0.1"

with:

local hpc_ips_ver=os.getenv("hpc_ips_ver") or "18.0.1.163"
local hpc_impi_ver=os.getenv("hpc_impi_ver") or "18.0.1"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants