Skip to content

build: update gt4py current state of main branch#150

Merged
FlorianDeconinck merged 1 commit into
NOAA-GFDL:developfrom
romanc:romanc/gt4py-update
May 27, 2025
Merged

build: update gt4py current state of main branch#150
FlorianDeconinck merged 1 commit into
NOAA-GFDL:developfrom
romanc:romanc/gt4py-update

Conversation

@romanc
Copy link
Copy Markdown
Collaborator

@romanc romanc commented May 27, 2025

Description

Updates gt4py to the current state of main. Notably, this includes

The full list of changes is here. This is in preparation for the next NDSL release, which is just around the corner.

How Has This Been Tested?

Covered by existing test suite.

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
  • New check tests, if applicable, are included: N/A

Notably, this includes

- refactor[cartesian|next] Move imports of implementation down one level #2045
- refactor[cartesian]: remove explicit use of boost #1971
- refactor[cartesian]: Remove __INLINED deprecation warning #2032

The full list of changes is
[here](GridTools/gt4py@1ba0a97...45324c8).
@romanc romanc requested a review from FlorianDeconinck May 27, 2025 14:56
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.

LGTM

@FlorianDeconinck FlorianDeconinck merged commit 28dd8b8 into NOAA-GFDL:develop May 27, 2025
5 checks passed
@romanc romanc deleted the romanc/gt4py-update branch May 28, 2025 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants