Skip to content

Comments

Muphys: Remove self-copy in else-branch of mask expressions#1069

Draft
edopao wants to merge 112 commits intomainfrom
muphy_remove_self_copies
Draft

Muphys: Remove self-copy in else-branch of mask expressions#1069
edopao wants to merge 112 commits intomainfrom
muphy_remove_self_copies

Conversation

@edopao
Copy link
Contributor

@edopao edopao commented Feb 19, 2026

No description provided.

fomics and others added 30 commits September 11, 2025 12:17
Clean up the graupel_only driver, and create an integration test to run through pytest. 

Co-authored-by: Will Sawyer <wsawyer@cscs.ch>
Co-authored-by: Will Sawyer <vectorflux@gmail.com>
Co-authored-by: Will Sawyer <wsawyer@cscs.ch>
@edopao
Copy link
Contributor Author

edopao commented Feb 19, 2026

cscs-ci run dace

@github-actions
Copy link

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.

@edopao
Copy link
Contributor Author

edopao commented Feb 19, 2026

cscs-ci run dace

@edopao
Copy link
Contributor Author

edopao commented Feb 20, 2026

R2B06 benchmark on latest icon4py:main (which includes the inout t field) and gt4py:v1.1.4.

Baseleine           vs  Self-copy Removal
0.8063652515411377      0.7960093021392822
0.8046061992645264      0.7870404720306396
0.800187349319458       0.8077375888824463
0.7698495388031006      0.8004043102264404
0.8041069507598877      0.7910645008087158
0.7722852230072021      0.7776751518249512
0.7911474704742432      0.7825534343719482
0.7928476333618164      0.7938387393951416
0.7766461372375488      0.7829110622406006
0.8041651248931885      0.8013942241668701

There is no clear improvement.

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.

5 participants