Skip to content

Expose stencil from gt4py#144

Merged
romanc merged 1 commit intoNOAA-GFDL:developfrom
romanc:romanc/expose-stencil
May 16, 2025
Merged

Expose stencil from gt4py#144
romanc merged 1 commit intoNOAA-GFDL:developfrom
romanc:romanc/expose-stencil

Conversation

@romanc
Copy link
Copy Markdown
Collaborator

@romanc romanc commented May 16, 2025

Description

We forgot to expose stencil from GT4Py in the new interface.

How Has This Been Tested?

Will be covered by pace tests once NOAA-GFDL/pace#114 is merged.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas: N/A
  • I have made corresponding changes to the documentation: N/A
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules: N/A
  • New check tests, if applicable, are included: N/A

@romanc romanc requested a review from CharlesKrop May 16, 2025 15:43
FlorianDeconinck
FlorianDeconinck approved these changes May 16, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@FlorianDeconinck FlorianDeconinck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most beautiful

@romanc romanc merged commit beb7cc7 into NOAA-GFDL:develop May 16, 2025
5 checks passed
@romanc romanc deleted the romanc/expose-stencil branch May 16, 2025 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@FlorianDeconinck FlorianDeconinck FlorianDeconinck approved these changes

@CharlesKrop CharlesKrop Awaiting requested review from CharlesKrop

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@romanc @FlorianDeconinck