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

Update Github runners to Ubuntu 24.04 #224

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

samhatfield
Copy link
Collaborator

No description provided.

@samhatfield samhatfield force-pushed the update_actions_os branch 15 times, most recently from 6601c88 to dcd870a Compare March 6, 2025 16:19
@samhatfield
Copy link
Collaborator Author

Note that the ectrans_test_adjoint test is broken for intel-llvm (ifx) due to a compiler bug. The following program behaves wrong when compiling with ifx and -fiopenmp:

PROGRAM REPRODUCER

IMPLICIT NONE

INTEGER, PARAMETER :: N = 5
INTEGER :: ARRAY(N)

ARRAY(:) = 1

PRINT *, "Values in PROGRAM"
PRINT *, ARRAY(1:N)
CALL TEST1

CALL TEST2(ARRAY)

CONTAINS

  SUBROUTINE TEST1
    PRINT *, "Values in TEST1"
    PRINT *, ARRAY(1:N)
  END SUBROUTINE TEST1

  SUBROUTINE TEST2(KVSET)
    INTEGER, INTENT(IN) :: KVSET(:)

    PRINT *, "Values in TEST2"
    PRINT *, KVSET
  END SUBROUTINE TEST2

END PROGRAM REPRODUCER

I've reported this to Jacob Poulsen at Intel.

@samhatfield samhatfield force-pushed the update_actions_os branch 3 times, most recently from 96742b4 to 80f7c00 Compare March 6, 2025 17:42
@samhatfield
Copy link
Collaborator Author

Good to merge.

@samhatfield samhatfield requested a review from wdeconinck March 6, 2025 18:02
@wdeconinck
Copy link
Collaborator

wdeconinck commented Mar 12, 2025

I would add the ifx compiler bug in a separate PR for visibility. This PR is then to be merged afterwards.

compiler_cxx: icpx
compiler_fc: ifx
# TODO: this tests is broken with intel-llvm (segfault)
ctest_options: -E ectrans_test_install
Copy link
Collaborator

Choose a reason for hiding this comment

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

ectrans_test_install fails??? That is very concerning...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, I'll reenable it so you can see the error.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

It seems that only the Fortran executables are working and the C executables transi_sptogp or transi_gptosp are causing a segfault. I wonder if we can try to reproduce this locally without a GitHub-runner.

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 had a go on Atos with the same version of Intel (2023.2.0) but the test passed. I think we'll need to debug this through the action itself.

@samhatfield
Copy link
Collaborator Author

See https://github.com/ecmwf-ifs/ectrans/pull/227/files for ifx bug fix.

@wdeconinck
Copy link
Collaborator

See https://github.com/ecmwf-ifs/ectrans/pull/227/files for ifx bug fix.

OK that is merged now. This can be rebased.

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.

2 participants