Skip to content

Conversation

@basava70
Copy link
Collaborator

@basava70 basava70 commented Jun 30, 2025

Phase 1: Changes from HPC and Data Processing to Improve PR #693

🎯 Goals

  • Convert init_tracers_AB subroutine to run completely on the GPU
    This subroutine has three calls to two internal subroutines:
    • tracer_gradient_elements
    • tracer_gradient_z
    • relax_to_clim
  • Run benchmarks and compare with Phase0 PR Openacc test main #693

… ACC ENTER DATA CREATE pragmas in fesom_module.F90
@basava70 basava70 marked this pull request as draft June 30, 2025 09:01
@basava70
Copy link
Collaborator Author

basava70 commented Jul 1, 2025

@suvarchal @JanStreffing @mandresm
I have a working and compiling state. I am running benchmarks and will update in the next comment.

@basava70 basava70 mentioned this pull request Jul 1, 2025
14 tasks
@basava70 basava70 marked this pull request as ready for review July 1, 2025 13:45
@basava70
Copy link
Collaborator Author

basava70 commented Jul 1, 2025

Benchmarks for Phase 1.
Since, 1 year simulation in inconclusive, do you want me to add 2 months and 4 months ?

Run Length Run Length Unit Steps per Day CPU (s) Phase0(s) Phase1(s)
1 d 32 1.806 125.2253 104.2
1 m 32 55.408 4160.025 3400.23
1 y 32 787.221 > 26400.0 > 26400.0
2 y 32 1287.22 NA NA

@JanStreffing
Copy link
Collaborator

Benchmarks for Phase 2. Since, 1 year simulation in inconclusive, do you want me to add 2 months and 4 months ?
Run Length Run Length Unit Steps per Day CPU (s) Phase1(s) Phase2(s)
1 d 32 1.806 125.2253 104.2
1 m 32 55.408 4160.025 3400.23
1 y 32 787.221 > 26400.0 > 26400.0
2 y 32 1287.22 NA NA

Gives enough of an idea as is IMO.

@basava70
Copy link
Collaborator Author

basava70 commented Jul 1, 2025

Benchmarks for Phase 2. Since, 1 year simulation in inconclusive, do you want me to add 2 months and 4 months ?
Run Length Run Length Unit Steps per Day CPU (s) Phase1(s) Phase2(s)
1 d 32 1.806 125.2253 104.2
1 m 32 55.408 4160.025 3400.23
1 y 32 787.221 > 26400.0 > 26400.0
2 y 32 1287.22 NA NA

Gives enough of an idea as is IMO.

Cool. I will leave it for now. You guys review and let me know if anything needs to be done. I will move on to Phase2 PR #734.

@suvarchal suvarchal merged commit a670959 into main Jul 1, 2025
7 checks passed
@JanStreffing JanStreffing deleted the openacc_Phase_1 branch July 1, 2025 14:34
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.

4 participants