Skip to content

Conversation

@philip-paul-mueller
Copy link
Contributor

@philip-paul-mueller philip-paul-mueller commented Apr 30, 2025

Instead of pulling directly from the official DaCe repo, we now (for the time being) pull from this PR.
This became necessary as we have a lot of open PR in DaCe and need some custom fixes (that can in their current form not be merged into DaCe).
In the long term however, we should switch back to the main DaCe repo.

The downside is that it is my personal fork.
@philip-paul-mueller philip-paul-mueller marked this pull request as ready for review April 30, 2025 10:11
@philip-paul-mueller philip-paul-mueller merged commit 9b9bf0f into GridTools:main Apr 30, 2025
31 checks passed
philip-paul-mueller added a commit that referenced this pull request May 7, 2025
…1913)

Due to the update to DaCe in
[PR#1976](spcl/dace#1976) the code generator is
now able to handle more Memlets directly as Cuda `memcpy()` calls. Thus
we have to change this function to reflect this case.
It even stands to reason that these functions can be removed altogether.

Note that the merge has become possible because of our switch to our
DaCe fork in [PR#2012](#2012).
stubbiali pushed a commit to stubbiali/gt4py that referenced this pull request Aug 19, 2025
Instead of pulling directly from the official DaCe repo, we now (for the
time being) pull from [this
PR](GridTools/dace#1).
This became necessary as we have a lot of open PR in DaCe and need some
custom fixes (that can in their current form not be merged into DaCe).
In the long term however, we should switch back to the main DaCe repo.
stubbiali pushed a commit to stubbiali/gt4py that referenced this pull request Aug 19, 2025
…ridTools#1913)

Due to the update to DaCe in
[PR#1976](spcl/dace#1976) the code generator is
now able to handle more Memlets directly as Cuda `memcpy()` calls. Thus
we have to change this function to reflect this case.
It even stands to reason that these functions can be removed altogether.

Note that the merge has become possible because of our switch to our
DaCe fork in [PR#2012](GridTools#2012).
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