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

Frontier build update #163

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

Conversation

nkoukpaizan
Copy link
Collaborator

@nkoukpaizan nkoukpaizan commented Jan 23, 2025

Merge request type

  • New feature
  • Resolves bug
  • Documentation
  • Other

Relates to

  • OPFLOW
  • SOPFLOW
  • SCOPFLOW
  • TCOPFLOW
  • CMake build system
  • Spack configuration
  • Manual
  • Web docs
  • Other

This MR updates

  • Header files
  • Source code
  • CMake build system
  • Spack configuration
  • Web docs
  • Manual
  • Other

Summary

  • This updates the Spack configuration on Frontier to use ROCm 6.3.1.
  • This also removes buildsystem setup for decommissioned OCLF computers (Ascent and Summit).
  • Deactivating PNNL mirror.
  • Updating GH actions.

@nkoukpaizan nkoukpaizan self-assigned this Jan 23, 2025
@nkoukpaizan nkoukpaizan marked this pull request as ready for review January 30, 2025 20:42
@pelesh pelesh self-requested a review January 30, 2025 20:46
Copy link
Collaborator

@pelesh pelesh left a comment

Choose a reason for hiding this comment

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

Built ExaGO from scratch without any problems. All but one test pass. The failing test is for experimental ExaGO code and the failure is unrelated to this PR.

59/59 Test #59: Python_pytest_MPI ............................................................   Passed    3.21 sec

98% tests passed, 1 tests failed out of 59

Label Time Summary:
incline-skip    =  60.96 sec*proc (2 tests)

Total Test time (real) = 336.95 sec

The following tests FAILED:
	 19 - FUNCTIONALITY_TEST_OPFLOW_RAJAHIOP_SPARSE_GPU_TOML_TESTSUITE (Failed)

I suggest we merge.

@pelesh pelesh mentioned this pull request Jan 31, 2025
19 tasks
@abhyshr
Copy link
Collaborator

abhyshr commented Feb 12, 2025

This mostly touches the buildsystem files so looks good to merge.

@nkoukpaizan
Copy link
Collaborator Author

This mostly touches the buildsystem files so looks good to merge.

I don't have the authorizations to merge this myself. Someone who does will have to merge.

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.

3 participants