Skip to content

ADR004-Storing of Grid files#38

Open
ninaburg wants to merge 6 commits intoecmwf:mainfrom
MeteoSwiss:adr004/storingOfGridFiles
Open

ADR004-Storing of Grid files#38
ninaburg wants to merge 6 commits intoecmwf:mainfrom
MeteoSwiss:adr004/storingOfGridFiles

Conversation

@ninaburg
Copy link

@ninaburg ninaburg commented Jan 20, 2026

Description

This PR adds ADR 004, documenting the architectural options for managing MeteoSwiss ICON grid definition files in Polytope (via eckit::geo).

It describes the current reliance on ECMWF-hosted services, evaluates alternative approaches, and outlines a preferred direction toward MeteoSwiss-managed grid storage, with open implementation questions.

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

@ninaburg ninaburg changed the title first commit ADR004-Storing of Grid files ADR004-Storing of Grid files Jan 20, 2026
@ninaburg ninaburg force-pushed the adr004/storingOfGridFiles branch from fa86862 to 4906bf2 Compare February 3, 2026 09:47
@ninaburg ninaburg marked this pull request as ready for review February 5, 2026 13:51
@ninaburg
Copy link
Author

ninaburg commented Feb 5, 2026

Dear @pmaciel and @jameshawkes, can I have your view on this ADR - Storing of Grid files we already discussed at the syncs? Thank you!

Copy link

@cfkanesan cfkanesan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice write up @ninaburg Thanks for putting this together, I still have some minor comments

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