forked from NOAA-GFDL/NDSL
-
Notifications
You must be signed in to change notification settings - Fork 0
Step 2: merge "release 2025.05.00" #2
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
Closed
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Markdown error fixes, formatting updates, and minor rewording of the document. Co-authored-by: Roman Cattaneo <[email protected]>
* No threshold / perturbation analysis by default
Currently, when translate tests fail, we run 10 (by default) times the
same translate test/savepoint again with slightly perturbed inputs. The
outputs are then analyzed to give developers an indication on whether
errors that they are seeing are in the expected range of "noise
propagation" or if this is more likely to be an error in porting.
* Rename {PACE -> NDSL}_TEST_N_THRESHOLD_SAMPLES
* Unrelated fix typos
- fix typos
- capitalize words at the start of a sentence
- add point a point at the end of a sentence
---------
Co-authored-by: Roman Cattaneo <[email protected]>
…NOAA-GFDL#126) * Ability to import gt4py tools from within NDSL. gt4py can no longer be imported independently when using NDSL, it must be run through NDSL to ensure proper precision * linting * added missing imports * added missing function * linting
* Ochestration with ConstantPropagation ConstantPropagation is part of the standard simplification pipeline. It was previously skipped because of an issue. In the meantime, that issue was resolved and we can propagate constants again. --------- Co-authored-by: Roman Cattaneo <[email protected]> Co-authored-by: Florian Deconinck <[email protected]>
* new constants * adding some stencils and the stefan-boltzman constant * adding namelist switches * add dtypes * update dtypes * hydlb * add triple point temp * cleanup of some constants * adding lsoil for lsm * add method to create array from compute domain * try ndim not 4d * iop * adding masked tridiag * added default stzrt to nd storage data * bool data handling * oops * added keyword to translate indices between Fortran and Python for translate tests * ndsl/stencils/testing/translate.py * remove break * adding triple point constant * update namelist * oops * better default * intermediate commmit * docstrings * linting --------- Co-authored-by: Roman Cattaneo <[email protected]>
…ntity (NOAA-GFDL#141) * Introduce `field` as a replacement for `quantity.view[:]` Rename `data_array` as `data_as_xarray` and introduce `field_as_xarray` * Unit tests * Fix usage of `data_array` * More fix * Litn * Add an halo read/write data/field test
* Adding sw_dynamics to NamelistDefaults * Linting NamelistDefaults comments * Add sw_dynamics to Namelist NOAA-GFDL#136 * retrigger checks
The function was replaced with `get_precision()` in 2025.03.00. Co-authored-by: Roman Cattaneo <[email protected]>
* Simple test for gt4py wrapper * Remove experimental features, which aren't in mainline yet --------- Co-authored-by: Roman Cattaneo <[email protected]>
Co-authored-by: Roman Cattaneo <[email protected]>
…L#130) * added keyword to translate indices between Fortran and Python for translate tests * ndsl/stencils/testing/translate.py * remove break * adding triple point constant * update namelist * oops * better default * updating namelist and adding some basic operations * linting and type safety * update labels for docs, added TODO
* Field Bundle functional code for 4D array Deferred type hint for data dimensions Unit tests * Fix imports * Fix `dtype` lacking in type registration Modernizing type hint Docstrings
* Expose area for 64-bit Make `ptop` available to GPU for orchestration * Lint
* Save in/out of stencils and decorated functions * Move `instrument` to debugger Verbose code * Log debugger on & config in debug * Move @Instrument once more * Add `track_by_parameter` and moved results in subdirectory * Simplify configuration * Minor fix to filename * Fix stencils save Check only when debugger present * Normalize dims naming * Use full qualified name when possible * Lint * Coarse documentation for debugger config * Protect catch-all for in xarray failures that don't raise properly * Config: allow for compute domain save in Quantity instead of full data field * Added tools to plot pyFV3 data * removed print statements * renamed folders * `plot_cube_sphere` public API * FV3 README license ackgnoledgement * Debugger knows how to serialize dataclasses Better handling of types unknown to xarray * Restore `Robinson` projection * Lint * Moar lint * Remove `DebugMode` Add warning for config not existing but set Use Pathlib for cross-OS resilience * Lint * `cartopy` import is not optional * Clear `__init__` * lint * Remove debug print * Use `field` instead of `view[:]` * Spelling --------- Co-authored-by: Charles Kropiewnicki <[email protected]>
* Update gt4py imports in example notebooks Follow-up from PR NOAA-GFDL#126 which adds an interface to gt4py as part of NDSL. * Change tests to import from ndsl.dsl.gt4py * Fix import from boilerplate --------- Co-authored-by: Roman Cattaneo <[email protected]>
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). Co-authored-by: Roman Cattaneo <[email protected]>
Co-authored-by: Roman Cattaneo <[email protected]>
Co-authored-by: Roman Cattaneo <[email protected]> Co-authored-by: Florian Deconinck <[email protected]>
Co-authored-by: Roman Cattaneo <[email protected]>
The one boost header that gt4py uses, comes now bundled with gt4py. We can thus remove boost (headers) from the list of dependencies. The docs were the only place left - as far as a quick search could see. Co-authored-by: Roman Cattaneo <[email protected]>
* refactor: xarray is not an optional import (anymore) * xarray: Fix checkpointer snapshots * gt4py is not an optional dependency * DaCe is not an optional dependency (anymore) * mpi4py is not an optional dependency (anymore) * Get optional imports in ndsl.optional_imports * Cleanup blank gt4py imports * remove #isort: skip --------- Co-authored-by: Roman Cattaneo <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Include a summary of the change and which issue is fixed. Please also include
relevant motivation and context. List any dependencies that are required for
this change.
Fixes # (issue)
If this is a hotfix to a released version, please specify it.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note
any relevant details for your test configuration (e.g. compiler, OS). Include
enough information so someone can reproduce your tests.
Checklist: