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.

Sign up for GitHub

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

Jump to bottom

Aurora config #2593

Open
wants to merge 93 commits into
base: master
Choose a base branch
from
Open

Aurora config #2593

wants to merge 93 commits into from

Conversation

oksanaguba
Copy link
Contributor

@oksanaguba oksanaguba commented Nov 6, 2023 •
edited
Loading

this branch will be eventually reworked into a branch with clean history

@oksanaguba oksanaguba changed the title WIP WIP spot config Nov 6, 2023
@E3SM-Autotester
Copy link
Collaborator

Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request.

@E3SM-Autotester
Copy link
Collaborator

Status Flag 'Pull Request AutoTester' - Failure: Timed out waiting for job SCREAM_PullRequest_Autotester_Mappy to start: Total Wait = 1803

  • Other jobs have been previously started - We must stop them...

oksanaguba and others added 24 commits November 7, 2023 16:34
@oksanaguba
wip cime changes
4f24ef4
@oksanaguba
sync ekat with its branch
18800c6
@oksanaguba
config, wip
ea9006e
@oksanaguba
typo
b8fe4ac
@oksanaguba
hardwire team size to 4
56c0016
@oksanaguba
kokkos prinf
0601c69
@oksanaguba
add sunspot-pvc
f425901
@oksanaguba
update ekat, ekat/kokkos
a8a9599
@oksanaguba
load certain cmake and python
09602cd
@oksanaguba
build issue fixed thatnks to Andrew and Daniel A
80bf0d5
@oksanaguba
Merge remote-tracking branch 'origin/master' into oksanaguba/spot
5fa0d1f 
 Conflicts:
	cime_config/machines/cmake_macros/oneapi-ifxgpu.cmake
	components/homme/CMakeLists.txt
	components/homme/cmake/HommeMacros.cmake
@oksanaguba
Merge remote-tracking branch 'E3SM/xyuan/e3sm_aurora' into oksanaguba…
8d288b2 
…/spot

 Conflicts:
	cime_config/machines/config_batch.xml
@oksanaguba
bad conflict resolution fixed
e2792ba
changes for aurora
735bac6
cache file
f5dfd58
wip
69b07b8
@oksanaguba
point to newest ekat changes
094798f
@oksanaguba
add auroracpu machine
8daf074
@oksanaguba
adding iostream
1c44b20
@oksanaguba
updated ekat branch
ca14fc6
@oksanaguba
bring changes from standalone homme sycl branch
1caf5dc
@oksanaguba
partial fix, adds escaped quotation marks
6bca905
@oksanaguba
debug statements
9f954b8
@oksanaguba
-g flags
f61ce74
@oksanaguba oksanaguba changed the title [WIP] spot config Aurora config Sep 6, 2024
rljacob
rljacob requested changes Sep 17, 2024
View reviewed changes
Copy link
Member

@rljacob rljacob left a comment

Choose a reason for hiding this comment

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

You should use/modify the aurora/sunspot configs added in E3SM-Project/E3SM#6553. This creates too many machine definitions and doesn't follow the conventions we've established for machines with cpu/gpu nodes.

@oksanaguba
Copy link
Contributor Author

@rljacob yes, though it may be a subsequent PR after this one goes in (but i will rename machines using scream suffix).

@rljacob
Copy link
Member

rljacob commented Sep 17, 2024

This PR can not go in as is. And adding "scream" is also not a solution.

@oksanaguba
update ekat (which has updated kokkos)
0a4d147
@oksanaguba
update aurora modules
49bd431
@oksanaguba
add no-vni
3afe043
@oksanaguba
Merge branch 'origin/master' into oksanaguba/spot
d01baa3 
 Conflicts:
	cime_config/machines/cmake_macros/oneapi-ifx.cmake
	cime_config/machines/cmake_macros/oneapi-ifxgpu.cmake
	cime_config/machines/cmake_macros/oneapi-ifxgpu_aurora.cmake
	cime_config/machines/config_batch.xml
	cime_config/machines/config_machines.xml
	components/eamxx/CMakeLists.txt
	components/eamxx/cmake/machine-files/aurora.cmake
	components/eamxx/src/dynamics/homme/eamxx_homme_iop.cpp
	externals/ekat
@oksanaguba
Copy link
Contributor Author

i cannot yet clean this branch till homme sycl changes are in scream repo.

@bartgol bartgol removed the AT: WIP label Nov 1, 2024
@bartgol bartgol force-pushed the master branch 4 times, most recently from 6318f41 to db5b35d Compare November 13, 2024 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rljacob rljacob rljacob requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@oksanaguba @E3SM-Autotester @rljacob @bartgol