-
Notifications
You must be signed in to change notification settings - Fork 20
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
Gt4py version 1015 #850
Gt4py version 1015 #850
Conversation
|
launch jenkins gt4py icon4py |
launch jenkins py-gt4py py-icon4py |
tsa🟢 unit test
🟢 integration test
|
balfrin🟢 unit test
🟢 integration test
🔴 system test
WARNING: Serial tests did not run for system tests |
balfrin🟢 unit test
🟢 integration test
🔴 system test
WARNING: Serial tests did not run for system tests |
daint🟢 unit test
🟢 integration test
🔴 system test
WARNING: Serial tests did not run for system tests |
daint🟢 unit test
🟢 integration test
🔴 system test
WARNING: Serial tests did not run for system tests |
launch jenkins py-gt4py |
tsa🟢 unit test
🟢 integration test
|
daint🟢 unit test
🟢 integration test
|
balfrin🟢 unit test
🟢 integration test
|
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.
Could you update the description here to say why the pytest args are being changed. Looks good, but someone with write access will need to approve.
Obviousely, when we don't add all the dependencies of |
In the end it is a package we solely use for exclaim. it is up to you to which extent it supports gt4py. But your change will reduce test coverage for all previous version. |
@mjaehn @dominichofer can you give formal approval? |
udpates to gt4py recipe:
As for the
py-icon4py
package in order to reduce the dependencies of thepy-gt4py
package by reduceing the tests being run to those relevant foricon-dsl
:gt4py
supports quite a few backends now: dace, gpu, gtfn, python buticon-dsl
uses thegtfn
backend for code generation throughicon4py
.code coverage has been removed from the test run, it is relevant for
gt4py
development but not for spack packaging.