-
Notifications
You must be signed in to change notification settings - Fork 48
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
feature[next] enable field origin in GTFN backend #1277
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.
2 more questions. I also pushed a change that enables a few more tests.
from gt4py.next.iterator import transforms | ||
|
||
if lift_mode != transforms.LiftMode.FORCE_INLINE: | ||
pytest.xfail("origin not yet supported in gtfn") |
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.
Can you fix the message? Do you understand what's not working?
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 don't really know, it crashes when freeing the temporaries, either bad logic or memory corruption. Probably also unrelated to this PR because it works when everything is inlined and the inputs and outputs are the same.
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.
Looks good!
Pass the origin of
LocatedField
s through the bindings to GridTools.