Hi @fprill, thank you so much for the contribution. It was very nice to read.

I have some comments on variable names for easier reading of the code (especially the big processing sequence), a technicality on the formatting of docstrings and I think two bigger comments.

One is on the iverbosity key, which I believe could be replaced/avoided.
The other is on potentially factoring the edge builders as they seem very similar in kind.

Again thanks for the great readability of the code, comments and style. (Didn't know about typeguard yet, I have to read up more on that. Seems quite useful.)

I marked some comments with nit as they're more nitpicks than actual blockers. Additionally, I think some of the larger comments can definitely be seen as a discussion basis rather than a blocker, let me know what you think.

Edit: I haven't left comments on a lot of nice sections in the code, as I didn't want to clutter the review even more. But just wanted to note that I did feel like there are a lot of really nice sections and it was a great read.

Hi @fprill, first of all thanks for the contributions and sorry for the late review. I leave here more general comments:

• In the last few days we have refactored a few things that may benefit this PR. We have now split the node/builders.py into different files under nodes/builders/, I think the icon nodes could go into their own ....py file. Would it be interesting to do the same with the edges?

• We also have a PR open for moving some optimising edge attributes builders. I would say it would be beneficial to move the edge_attrs defined in icon_mesh.py to edges/attributes.py. I think for edge direction you can rely on the already implemented and move the edge_length to a new ArcLength(). In PR #54 , we have already seen huge performance improvements and in the future we plan to go further to allow parallelisation with different devices. The differences are a few seconds now, but it could play a more important role in higher resolutions.

• Other useful improvements would be to add a test for node & edge builders to avoid breaking functionality with other changes in the future without noticing and to add a docs/graphs/node_coordinates/icon_mesh.rst page with the information you consider of interest.

Thanks again for all the work, and I am free to chat to move the PR forward.

Thank you for your contribution!