Skip to content

Conversation

@lukasm91
Copy link
Contributor

@lukasm91 lukasm91 commented Oct 7, 2025

Currently, we allocate FIELDG only to call INIT_FIELDG, which itself is used exclusively for computing the XLON and YLAT arrays. We also deallocate FIELDG within the same routine.

Since the calculation of these arrays is straightforward and does not require any global data, this PR proposes to compute them directly. A further improvement could be to move the XLON/YLAT computation from INIT_FIELDG into a dedicated function to avoid duplicating this code.

@awnawab
Copy link
Collaborator

awnawab commented Oct 9, 2025

Hey @lukasm91. Thanks a lot for this, it looks like a very sensible change to reduce memory consumption. The CI is currently fubar'd, once #110 is merged I'd like to rebase this over develop and let the CI run through, but essentially this change looks good 👌

@awnawab awnawab changed the base branch from main to develop October 9, 2025 09:43
@awnawab awnawab merged commit 3c22dcb into ecmwf-ifs:develop Oct 10, 2025
15 checks passed
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.

2 participants