Conversation
Added new references and updated existing ones in the index.
giordano
reviewed
Feb 16, 2026
| 1. Vankevich, R. Ye., Rodionov, А. А., Shpilev, N. N., and Chebotkova, V. V. (2025) [Modeling the origin and evolution of convective vortex structures on a slope. Numerical experiment](https://doi.org/10.59887/2073-6673.2025.18(4)-2), _Fundamental and Applied Hydrophysics_, **18(4)**, 20-27. DOI: [10.59887/2073-6673.2025.18(4)-2](https://doi.org/10.59887/2073-6673.2025.18(4)-2) | ||
| 1. Vankevich, R. Y., Rodionov, А. А., Shpilev, N. N., and Chebotkova, V. V. (2025) [Modeling the origin and evolution of convective vortex structures on a slope. Numerical experiment](https://doi.org/10.59887/2073-6673.2025.18(4)-2), _Fundamental and Applied Hydrophysics_, **18(4)**, 20-27. (in Russian) DOI: [10.59887/2073-6673.2025.18(4)-2](https://doi.org/10.59887/2073-6673.2025.18(4)-2) | ||
|
|
||
| 1. Vankevich, R. E., Isaev, A. V., and Ryabchenko, V. A. (2025) [On approaches to improving the description of North Sea water inflows in the general circulation models of the Baltic Sea](10.59887/2073-6673.2025.18(3)-3), _Fundamental and Applied Hydrophysics_, **18(3)**, 37-52. (in Russian) DOI: [10.59887/2073-6673.2025.18(3)-3](https://doi.org/10.59887/2073-6673.2025.18(3)-3) |
Collaborator
There was a problem hiding this comment.
This broke the docs build 🥲
glwagner
referenced
this pull request
Feb 20, 2026
* Support boolean comparison operations on fields and infer eltype from operations Register comparison operators (>, <, >=, <=) as binary operations on fields, and use Base.promote_op to infer the result type of all AbstractOperation constructors instead of hardcoding eltype(grid). Field(op) now allocates data matching the operation's eltype, so boolean operations produce Bool arrays. Closes #5307 Co-Authored-By: Claude Opus 4.6 <[email protected]> * Fix doctest and operator precedence in comparison tests - Suppress Set output in @binary doctest (non-deterministic ordering) - Wrap comparison `isa` tests in parentheses to fix precedence (`u > v isa T` parses as `u > (v isa T)`, not `(u > v) isa T`) Co-Authored-By: Claude Opus 4.6 <[email protected]> * Fall back to eltype(grid) when promote_op fails for KernelFunctionOperation Base.promote_op can return Union{} or non-concrete types (like Any) for complex kernel functions (e.g. ForcingKernelFunction, buoyancy_perturbation). This caused MethodErrors when trying to allocate data arrays. Fall back to eltype(grid) when promote_op cannot infer a concrete return type. Co-Authored-By: Claude Opus 4.6 <[email protected]> * Bump ambiguity threshold from 345 to 355 for comparison operators Adding >, <, >=, <= as binary operations on fields introduces 10 new method ambiguities with Base's definitions. Co-Authored-By: Claude Opus 4.6 <[email protected]> * Refactor operation constructors: accept T as parameter, infer at higher level - All operation inner constructors now accept `::Type{T}` as an optional final argument with a sensible default, making them clean passthroughs - BinaryOperation, UnaryOperation, MultiaryOperation: default T via `Base.promote_op` (reliable for simple eltypes) - KernelFunctionOperation inner constructor takes `(kernel_function, grid, arguments::Tuple, T=eltype(grid))` — no argument splatting, usable as internal interface. Outer convenience constructor splats + infers T. - adapt_structure, on_architecture, and multi-region getregion pass `eltype(op)` explicitly, avoiding redundant re-inference Co-Authored-By: Claude Opus 4.6 <[email protected]> * Remove promote_op from KernelFunctionOperation constructor Kernel functions are too complex for Base.promote_op to reliably infer return types. For example, SeawaterPolynomials.ρ with a Float32 grid but Float64 EOS coefficients causes promote_op to return Float64, changing the KFO eltype from Float32 to Float64 and breaking Float32 seawater density tests. Revert to always using eltype(grid) for KFO, as on main. The promote_op eltype inference is still used for Binary, Unary, and Multiary operations where it works reliably. Co-Authored-By: Claude Opus 4.6 <[email protected]> * Remove promote_op from KFO outer constructor, use eltype(grid) default promote_op inferred Float64 for TEOS10 density on Float32 grids (because EOS polynomial coefficients are Float64), causing precision mismatches in seawater density tests. The inner constructor already defaults to eltype(grid); users can override T explicitly via the inner constructor when needed. Co-Authored-By: Claude Opus 4.6 <[email protected]> * Retrigger CI --------- Co-authored-by: Claude Opus 4.6 <[email protected]>
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
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.
Added new references and updated existing ones in the index.