Skip to content

Conversation

@philip-paul-mueller
Copy link
Contributor

@philip-paul-mueller philip-paul-mueller commented Jun 5, 2025

Updates the DaCe dependency.

Note:
This PR includes blocked_copy_to_map PR, which makes the silent transformation of Maps inside the dace code generator a hard error.
This is the actual correct behaviour at least in GT4Py as this leads to performance degradation and even launch errors.

There is one PR missing, that is not yet opened in DaCe.
However, that PR is not needed by GT4Py, thus we can technically merge it.
@edopao edopao changed the title build[dace][nex]: Updated DaCe build[dace][next]: Updated DaCe Jun 5, 2025
edopao
edopao previously approved these changes Jun 5, 2025
Now the generation of Maps in the code generator in GPU mode has become a hard error.
This makes it imperative that we only merge the PR once we have CI back and have made sure that all tests, also the one in ICON4Py have passed!
@philip-paul-mueller philip-paul-mueller changed the title build[dace][next]: Updated DaCe DO NOT MERGE build[dace][next]: Updated DaCe Jun 6, 2025
@edopao edopao dismissed their stale review June 6, 2025 11:34

Requires more testing, because of GridTools/dace#5

@philip-paul-mueller philip-paul-mueller changed the title DO NOT MERGE build[dace][next]: Updated DaCe build[dace][next]: Updated DaCe Jun 10, 2025
@philip-paul-mueller philip-paul-mueller merged commit e3d0790 into GridTools:main Jun 10, 2025
30 of 31 checks passed
stubbiali pushed a commit to stubbiali/gt4py that referenced this pull request Aug 19, 2025
Updates the DaCe dependency.

This updates does not contains the updates to the GPU codegen.
They are handled in a separate [PR#2070](GridTools#2070).
@philip-paul-mueller philip-paul-mueller deleted the dace_dependency_2025-06-05 branch October 9, 2025 05:58
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.

2 participants