Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

don't display bad practice #746

Merged
merged 4 commits into from
Jun 21, 2024
Merged

don't display bad practice #746

Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
4624a0a
don't display bad practice
wxtim Jun 20, 2024
549da79
add warning about using directives equivelent to ETL
wxtim Jun 21, 2024
9ffb2b6
Rearrange execution time limit admonition
MetRonnie Jun 21, 2024
898d3c6
Merge pull request #11 from MetRonnie/time-limit-directives
wxtim Jun 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions src/user-guide/task-implementation/job-submission.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,15 @@ characters using
Directives Section Quirks (PBS, SGE, ...)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

.. warning::

Use :cylc:conf:`[runtime][<namespace>]execution time limit` rather
than directives.

This makes Cylc aware of execution time limits so that execution
time out can be identified even if the :term:`job runner` cannot be
contacted.

To specify an option with no argument, such as ``-V`` in PBS or
``-cwd`` in SGE you must give a null string as the directive value in
the :cylc:conf:`flow.cylc` file.
Expand All @@ -70,7 +79,7 @@ line:

.. code-block:: none

-l=select=28:ncpus=36:mpiprocs=18:ompthreads=2:walltime=12:00:00
-l=select=28:ncpus=36:mpiprocs=18:ompthreads=2

(Left hand side is ``-l``. A second ``-l=...`` line will override the first.)

Expand All @@ -82,7 +91,6 @@ Or separate the items:
-l ncpus=36
-l mpiprocs=18
-l ompthreads=2
-l walltime=12:00:00

.. note::

Expand Down
Loading