-
Notifications
You must be signed in to change notification settings - Fork 49
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
feat[next]: Pass sizes to temporaries from gt4py program #1359
feat[next]: Pass sizes to temporaries from gt4py program #1359
Conversation
…oraries' into pass-sizes-to-temporaries # Conflicts: # src/gt4py/next/program_processors/codegens/gtfn/gtfn_module.py # tests/next_tests/integration_tests/feature_tests/ffront_tests/test_execution.py
@DropD Would you please review
|
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.
We'll probably need another round. There's still some code duplication that I would like to get rid of before I take a deeper look.
tests/next_tests/integration_tests/feature_tests/ffront_tests/test_execution.py
Outdated
Show resolved
Hide resolved
tests/next_tests/integration_tests/feature_tests/ffront_tests/test_execution.py
Outdated
Show resolved
Hide resolved
@@ -19,9 +19,12 @@ | |||
import pytest |
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.
I'll revisit the changes here in a later review round.
…gt4py into pass-sizes-to-temporaries
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.
The program_processor submodules look ok.
* update code generation backend to gt4py changes after [PR](GridTools/gt4py#1359) * fix mypy
* update code generation backend to gt4py changes after [PR](GridTools/gt4py#1359) * fix mypy
Description
Requirements
If this PR contains code authored by new contributors please make sure:
AUTHORS.md
file adding the names of all the new contributors.