Skip to content

don't merge: workaround caching issue#2425

Closed
havogt wants to merge 3 commits intomainfrom
workaround_caching_issue_with_embedded_inverse_image_caching
Closed

don't merge: workaround caching issue#2425
havogt wants to merge 3 commits intomainfrom
workaround_caching_issue_with_embedded_inverse_image_caching

Conversation

@havogt
Copy link
Copy Markdown
Contributor

@havogt havogt commented Dec 18, 2025

The NDArrayConnectivityField stores the inverse image for embedded in a _cache property which is part of the hash for lowering. This hash changes on each (interpreter) run, therefore the cache is not properly re-used.

We should find a more general solution than this workaround... next year...

@kotsaloscv
Copy link
Copy Markdown
Contributor

FYI -> eth-cscs/cscs-reframe-tests#493
By patching on-the-fly GT4Py, the cache works as expected.

@egparedes
Copy link
Copy Markdown
Contributor

egparedes commented Mar 17, 2026

PR #2536 should fix the original issue in the proper way.

@havogt havogt closed this Mar 19, 2026
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