MetOffice / MetOffice/lfric_apps

Additional PC2 optimisations for NG-ARCH

Open
#47 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Fortran
Stars
31
Forks
118
Avg merge
2d 4h
Merged PRs (30d)
8

Description

### Contact Details

@tinyendian

### Are there any linked Issues or Pull Requests?

ticket:1046
#53

### Brief description

This issue covers additional PC2 optimisations for the NG-ARCH project. The following subroutines will be optimised by adding OpenMP and compiler directives using PSyclone transmute scripts:
* `pc2_bm_initiate`
* `pc2_bl_forced_cu`
* `pc2_initiation_ctl`

### Further details of the issue.

Subroutine `pc2_bm_initiate` was originally targeted in TRAC ticket [#942](https://code.metoffice.gov.uk/trac/lfric_apps/ticket/942), but the PSyclone transmute script had to be excluded when the changes were merged as the removal of compiler directives through PSyclone transmutation led to KGO comparison failures.

This issue was subsequently addressed by TRAC ticket [#1046](https://code.metoffice.gov.uk/trac/lfric_apps/ticket/1046), which also includes the removal of an OpenMP Single region. The TRAC ticket will be completed with this GitHub issue, and extended with two additional PC2 optimisations to complete the NIWA/ESNZ contributions to NG-ARCH.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with the PSyclone transmute scripts for pc2_bm_initiate, pc2_bl_forced_cu, and pc2_initiation_ctl, then review the KGO comparison failures described for TRAC ticket 942. Apply the planned OpenMP and compiler-directive changes, and confirm that the relevant KGO comparisons pass without regressions.

Written by the indexing model from the issue text.

Assessment

Tech stack
fortran
Domain
hpc, performance
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.