Skip to content

Conversation

@nefrathenrici
Copy link
Member

@nefrathenrici nefrathenrici commented Sep 17, 2025

Purpose

Use ClimaCore's CommonGrids and CommonSpaces to create spaces.

To-do

Content

  • Added new grids.jl file with grid constructors: SphereGrid, ColGrid, BoxGrid, PlaneGrid. Topography is handled by a function hypsography_function_from_topography which returns a hypsography function passed to ClimaCore.
  • New get_spaces(grid, context) function replaces parsed-args dependent get_spaces(parsed_args, params, comms_ctx)
  • Removed old space creation functions: periodic_line_mesh, periodic_rectangle_mesh, make_hybrid_spaces
  • Replaced CTh (horizontal contravariant) with CT12
  • Updated ∂ᶜK_∂ᶜuₕ field type from DiagonalMatrixRow{Adjoint{FT, CTh{FT}}} to use CT12
  • Updated g³ʰ() to return explicit CT12 components using SMatrix
  • Updated is_column and do_dss to match new spaces
  • Updated ci_plots.jl and PrecompileCI
  • Updated gravity wave tests and other test helpers

  • I have read and checked the items on the review checklist.

@nefrathenrici nefrathenrici force-pushed the ne/grids branch 2 times, most recently from a878a51 to 1342d7b Compare November 5, 2025 18:18
@nefrathenrici nefrathenrici force-pushed the ne/grids branch 3 times, most recently from c1b1a4d to 725681b Compare November 21, 2025 18:27
@nefrathenrici nefrathenrici force-pushed the ne/grids branch 4 times, most recently from 7327a63 to 3fbfbfe Compare December 2, 2025 18:25
@nefrathenrici nefrathenrici marked this pull request as ready for review December 2, 2025 20:59
@nefrathenrici nefrathenrici force-pushed the ne/grids branch 3 times, most recently from 4fe3d75 to 4b02236 Compare December 2, 2025 23:18
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