-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: Greenline metrics coefficients aj #429
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of them are simple stencil calculation that compute regular (not sparse fields) like Field[[CellDim, KDim], wpfloat]
or Field[[EdgeDim, KDim], wpfloat]
there is no reason to do this in numpy instead of gt4py.
Also, thinks about where you put the functions, either put everything below common/metrics/
or the more general functions like the gradients in common/math
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/tests/interpolation_tests/test_interpolation_fields3.py
Outdated
Show resolved
Hide resolved
…lation_coefficients3_aj
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/tests/interpolation_tests/test_interpolation_fields3.py
Outdated
Show resolved
Hide resolved
…ion_fields3.py Co-authored-by: Magdalena <luzm@ethz.ch>
…_interpolation_coefficients3_aj
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly moving around things: Try to get rid of the interpolation_fields3.py
and its test file.
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/src/icon4py/model/common/interpolation/interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/tests/interpolation_tests/test_interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/tests/interpolation_tests/test_interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/tests/interpolation_tests/test_interpolation_fields3.py
Outdated
Show resolved
Hide resolved
model/common/tests/interpolation_tests/test_interpolation_fields3.py
Outdated
Show resolved
Hide resolved
7928040
to
53c5ac6
Compare
model/common/src/icon4py/model/common/interpolation/interpolation_fields.py
Outdated
Show resolved
Hide resolved
cscs-ci run default |
launch jenkins spack |
model/common/src/icon4py/model/common/interpolation/interpolation_fields.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Magdalena <luzm@ethz.ch>
Co-authored-by: Magdalena <luzm@ethz.ch>
cscs-ci run default |
launch jenkins spack |
…_interpolation_coefficients3_aj
Mandatory Tests Please make sure you run these tests via comment before you merge!
Optional Tests To run benchmarks you can use:
To run tests and benchmarks with the DaCe backend you can use:
In case your change might affect downstream icon-exclaim, please consider running
For more detailed information please look at CI in the EXCLAIM universe. |
cscs-ci run default |
launch jenkins spack |
first metric coefficients