Skip to content

On atlas-interpolation tool and 'Output and visualisation'#5

Open
sbrdar wants to merge 16 commits into
ecmwf:mainfrom
sbrdar:fix/atlas_tag_my
Open

On atlas-interpolation tool and 'Output and visualisation'#5
sbrdar wants to merge 16 commits into
ecmwf:mainfrom
sbrdar:fix/atlas_tag_my

Conversation

@sbrdar
Copy link
Copy Markdown
Contributor

@sbrdar sbrdar commented May 12, 2026

Description

  • fix the break of the Pelican metadata block
  • create a better dependency for rebuilding the pages
  • add Tool >> atlas-interpolation, and Design >> Output and Visualisation

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.

Comment thread content/tools/atlas-grids.rst Outdated
require : atlas

- atlas-fesom :
git : ${BITBUCKET}/~nawd/atlas-fesom
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
git : ${BITBUCKET}/~nawd/atlas-fesom
git : ${GITHUB}/ecmwf/atlas-fesom

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Corrected. Thanks! Still the problem is that people may not use the atlas-bundle script.

.. note-warning::

Gmsh is great to visualize low resolution meshes but becomes sluggish to unresponsive for high resolution meshes.
This tool is however invaluable for development and for understanding of how low resolution grids and meshes are constructed.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think it was OK to leave this.

Copy link
Copy Markdown
Contributor Author

@sbrdar sbrdar May 18, 2026

Choose a reason for hiding this comment

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

I have moved this sentence to Design >> Output and Visualisation, because under Tools >> atlas-meshgen it does not make much sense. The Tools pages are not about Gmsh.
Let me know if you still want that warning repeated under atlas-meshgen.

Comment on lines +22 to +25
.. code :: bash

$ OMP_NUM_THREADS=1 mpirun -np 4 atlas-interpolation --s.grid <sgrid> --t.grid <tgrid> --interpolation <interpolation> \
[--output-matrix|--read-matrix|--test-matrix] [--format scrip|eckit] [--output-gmsh]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

it is not needed to add the OMP_NUM_THREADS=1 mpirun -np 4 here, but it can be further that parallelisation may be achieved via OpenMP multithreading (export OMP_NUM_THREADS=...) and distributed memory parallelisation via mpirun, srun etc.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have removed the parallelisation from the line 24, and only mentioned it in the text below.

Comment thread content/tools/atlas-interpolation.rst Outdated
$ atlas-interpolation --help

It is worth noting that some other interpolation methods available in Atlas can be
better suited (see `ECMWF documentation <https://confluence.ecmwf.int/x/ITsbHg>`_).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this page publicly accessible? Should it be?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have copied over the content from the restricted page. The diffusion properties and timings of the Atlas operators are now available to the public.

@sbrdar sbrdar requested a review from wdeconinck May 19, 2026 09:15
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