Skip to content

Commit 6945afa

Browse files
dennisYatuninclaude
andcommitted
Restore Fields.ColumnField as an alias of FiniteDifferenceField
The rewrite removed ColumnField (a Field with DataColumn values), but ClimaAtmos main uses it as a type annotation, so ClimaAtmos and everything depending on it (ClimaCoupler) fail to precompile against this branch. Single-column fields are now identified by their space, so the old ColumnField is the same set of fields as FiniteDifferenceField: FD fields and column slices of extruded fields match the alias, full 3-D fields do not. Verified that ClimaAtmos main precompiles and loads against a patched worktree. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 48ac9e2 commit 6945afa

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

src/Fields/Fields.jl

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,12 @@ const SpectralElementField2D{V, S} =
6464

6565
const FiniteDifferenceField{V, S} =
6666
Field{V, S} where {V <: DataLayout, S <: Spaces.FiniteDifferenceSpace}
67+
# Backwards-compatibility alias for the pre-rewrite ColumnField, which was a
68+
# Field with DataLayouts.DataColumn values. Single-column fields are now
69+
# identified by their space (a FiniteDifferenceSpace, which may wrap a
70+
# ColumnGrid returned by `column(::ExtrudedFiniteDifferenceSpace, ...)`), so
71+
# the old ColumnField is the same set of fields as FiniteDifferenceField.
72+
const ColumnField = FiniteDifferenceField
6773
const FaceFiniteDifferenceField{V, S} =
6874
Field{V, S} where {V <: DataLayout, S <: Spaces.FaceFiniteDifferenceSpace}
6975
const CenterFiniteDifferenceField{V, S} = Field{

0 commit comments

Comments
 (0)