Skip to content

DO NOT MERGE: Check Broadcast Node#1240

Draft
philip-paul-mueller wants to merge 10 commits into
C2SM:mainfrom
philip-paul-mueller:phimuell_test_fill_node
Draft

DO NOT MERGE: Check Broadcast Node#1240
philip-paul-mueller wants to merge 10 commits into
C2SM:mainfrom
philip-paul-mueller:phimuell_test_fill_node

Conversation

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator

Is related to GT4Py PR#2386.

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run extra

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run dace

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run extra

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run dace

@philip-paul-mueller

philip-paul-mueller commented May 6, 2026

Copy link
Copy Markdown
Collaborator Author

Results using GT4Py 3dcf818b4af5 indicating that there is some performance degradation.

bench_blueline_stencil_compute

Also the failures of teh CI are due to timeouts.

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

It is also interesting that the transformation is not really integrated yet.
Currently the node is at the beginning expanded, this should generate the same SDFG as if we would lower it to a Map.

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run extra

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run dace

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run dace

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run extra

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run dace

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

Here are the new data, it got better, but there are still some degradation.

bench_blueline_stencil_compute

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

1 similar comment
@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run dace

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

Newest results.
I would say the degradation is because of broadcast loops that were not integrated into other stencils

bench_blueline_stencil_compute

@github-actions

Copy link
Copy Markdown

Mandatory Tests

Please make sure you run these tests via comment before you merge!

  • cscs-ci run default
  • cscs-ci run distributed

Optional Tests

To run benchmarks you can use:

  • cscs-ci run benchmark-bencher

To run tests and benchmarks with the DaCe backend you can use:

  • cscs-ci run dace

To run test levels ignored by the default test suite (mostly simple datatest for static fields computations) you can use:

  • cscs-ci run extra

For more detailed information please look at CI in the EXCLAIM universe.

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run default

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run distributed

@philip-paul-mueller

Copy link
Copy Markdown
Collaborator Author

cscs-ci run dace

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.

1 participant