Use `todo` Sphinx directive more.
authorM. Eric Irrgang <ericirrgang@gmail.com>
Thu, 26 Dec 2019 15:45:59 +0000 (18:45 +0300)
committerEric Irrgang <ericirrgang@gmail.com>
Thu, 2 Jan 2020 13:53:14 +0000 (14:53 +0100)
Start making "to do"s more actionable. Use the Sphinx `todo` directive
to allow special processing. For comment blocks that imply planned
activity, convert comment to `todo` directive for more easily discovered
and tracked doctree elements.

"To do" items should have a clear call to action and some follow through,
such as being prohibited in release branches or at least referring to
tracked issues, but that is beyond the scope of this change.
This change only seeks to make the mass of "to do"s more discoverable.

Change-Id: I8cbc06a5a1d78791bc6e9b0a2530632ad09a9380

16 files changed:
docs/dev-manual/build-system.rst
docs/dev-manual/jenkins.rst
docs/dev-manual/overview.rst
docs/dev-manual/style.rst
docs/dev-manual/tools.rst
docs/index.rst
docs/reference-manual/file-formats.rst
docs/reference-manual/index.rst
docs/reference-manual/preface.rst
docs/user-guide/environment-variables.rst
docs/user-guide/faq.rst
docs/user-guide/force-fields.rst
docs/user-guide/index.rst
docs/user-guide/mdp-options.rst
docs/user-guide/mdrun-performance.rst
docs/user-guide/run-time-errors.rst

index 0b0591138d81a04109dd7ef376db4d35c0d4a74f..a98c9e9f84298b72f2c7155d6200c3565e6981b7 100644 (file)
@@ -111,10 +111,14 @@ This section provides a (currently incomplete) list of cache variables that
 developers or advanced users can set to affect what CMake generates and/or what
 will get built.
 
-.. TODO: Figure out where to document basic variables intended for user
+.. todo::
+
+   Figure out where to document basic variables intended for user
    consumption, and how does it relate to documentation here.
 
-.. TODO: Document the remaining variables below, and identify any variables
+.. todo::
+
+   Document the remaining variables below, and identify any variables
    missing from the list.
 
 Compiler flags
@@ -170,7 +174,7 @@ Variables affecting compilation/linking
    Defaults to ``OFF``, and there should not be any need to change this in a
    manual build.
 
-   .. TODO: This could likely be replaced by a (yet another) build type.
+   .. todo:: This could likely be replaced by a (yet another) build type.
 
 .. cmake:: GMX_BUILD_MDRUN_ONLY
 
@@ -388,7 +392,7 @@ Variables affecting special targets
    If ``OFF`` (the default), all detection is skipped and the manual cannot be
    built.
 
-   .. TODO: Consider if this is really necessary, or if we could just use
+   .. todo:: Consider if this is really necessary, or if we could just use
       GMX_DEVELOPER_BUILD.
 
 .. cmake:: GMX_BUILD_TARBALL
@@ -442,7 +446,9 @@ Variables affecting special targets
 External libraries
 ------------------
 
-.. TODO: List external libraries used (either from src/external/, or from the
+.. todo::
+
+   List external libraries used (either from src/external/, or from the
    system), whether they are required or optional, what functionality they
    provide for Gromacs, and how to control their use.
 
index 09ac69d3833faaad63048ad2c89baed88fece952..8ff99a0923d536445ad46e3327055638bf0680e0 100644 (file)
@@ -15,7 +15,7 @@ Separate page documents how to interact with the Jenkins UI for these builds:
 :doc:`releng/jenkins-howto` has information on how to do common things with
 Jenkins builds.
 
-.. TODO: Add a link to a wiki page about general Jenkins documentation, once
+.. todo:: Add a link to a wiki page about general Jenkins documentation, once
    there is more of that.
 
 Pre-submit verification
index 4a63fd4cf0242953855e2fccf522e0167cc46086..e9bf4c8baa7d5a67cd370c51718ec9191ca3953a 100644 (file)
@@ -274,7 +274,7 @@ Doxygen documentation
   See :doc:`doxygen` for details of how the Doxygen documentation is built and
   organized.
 
-.. TODO: Create a separate page (at the front of the developer guide, and/or at
+.. todo:: Create a separate page (at the front of the developer guide, and/or at
    the main index.rst) that describes the documentation from readers'
    perspective, and move relevant content there.  This should contain just an
    overview of how the documentation is organized in the source tree.
index 6ef21feadb473f0511d2c432037eceab93a2ce3c..761a455524c3d5f00f19229d6f795f798426001c 100644 (file)
@@ -35,4 +35,4 @@ this page.
 :doc:`commitstyle`
   Guidelines for formatting git commits when sending in proposed fixes for code review.
 
-.. TODO: Add more guidelines
+.. todo:: Add more guidelines
index 8c9dd91a30af8b9b2becc216f79c6597abd94306..1d19150461322f699021ae20a8d169d3a67744a9 100644 (file)
@@ -62,7 +62,7 @@ Redmine
 Build system
 ------------
 
-.. TODO: details, ASAN, others?
+.. todo:: details, ASAN, others?
 
 CMake
   Main tool used in the build system.
index 65bd93cfbabaca3995d16cd67ad7c01b73d99c60..002cbe52cf9e560b8a7aadc3538ac6609ee13b11 100644 (file)
@@ -2,7 +2,9 @@
 Welcome to the |Gromacs| documentation!
 =======================================
 
-..  TODO : consolidate at least some of the material in the
+..  todo::
+
+    Consolidate at least some of the material in the
     Documentation links below into the new user guide, along with all
     of http://www.gromacs.org/Documentation/Cut-off_schemes,
     http://www.gromacs.org/Documentation/Acceleration_and_parallelization
index 5567d3a7e45828950bedc61b6c43e27f16dab7fd..28ecdc9347c324306c6c2ae97c6cb8bc62239ac4 100644 (file)
@@ -1,7 +1,7 @@
 File formats
 ============
 
-.. TODO in future patch: update for accuracy, organize better, improve formatting
+.. todo:: in future patch: update for accuracy, organize better, improve formatting
 
 Summary of file formats
 ^^^^^^^^^^^^^^^^^^^^^^^
index 44ac3bbf1c583576c11176af31593b5011b7af9b..6ef673093a782b74f98ea619d3b3d34ec5211c88 100644 (file)
@@ -6,7 +6,7 @@ Reference Manual
 
 .. highlight:: bash
 
-.. TODO this needs to be carefully checked that I didn't mess anything up too bad
+.. todo:: this needs to be carefully checked that I didn't mess anything up too bad
 
 .. ifconfig:: gmx_image_convert == 'possible'
 
index 66dd6b5982eb44ebc99d3a59ed5acbd16b50ebca..b583b4df968472d953edc298c1a701b271d31fa6 100644 (file)
@@ -49,7 +49,7 @@ minor release number as your |Gromacs| installation.
 Citation information
 --------------------
 
-.. TODO needs link to ref list
+.. todo:: needs link to ref list
 
 |GMX_MANUAL_DOI_STRING|
 
index 5d8f77e59dc4cdce31db2f935ec3aa5d19709229..4b68280c7f179d20e98048986601cd56088aaaf4 100644 (file)
@@ -4,7 +4,7 @@
 .. Another useful one-liner to find undocumentedvariables:
 ..  ( export INPUT_FILE=docs/user-guide/environment-variables.rst; GIT_PAGER="cat ";   for ss in `for s in $(git grep getenv |  sed 's/.*getenv("\(.*\)".*/\1/' | sort -u  | grep '^[A-Z]'); do [ $(grep $s $INPUT_FILE -c) -eq 0 ] && echo $s; done `; do git grep $ss ; done )
 
-.. TODO: still undocumented GMX_QM_GAUSSIAN_NCPUS
+.. todo:: still undocumented GMX_QM_GAUSSIAN_NCPUS
 
 Environment Variables
 =====================
index 352681430aa077e109c5801e1d6e27280f004bb9..2c45196c27e0495bc95bb988acaa547a86e5bf4b 100644 (file)
@@ -111,7 +111,9 @@ Questions regarding simulation methodology
 
     You can choose different values for :mdp:`tinit` and :mdp:`init-step`.
 
-    .. TODO make links work :ref:`Continuing simulations <gmx-cont-simulation>`.
+    .. todo:: Add "Continuing simulations" content (label: gmx-cont-simulation) and link.
+
+        e.g. ``:ref:`Continuing simulations <gmx-cont-simulation>`.``
 
 #.  Why can't I do conjugate gradient minimization with constraints?
 
@@ -132,16 +134,24 @@ Questions regarding simulation methodology
     You can either prepare a new :ref:`mdp` file, or extend the simulation time
     in the original :ref:`tpr` file using :ref:`convert-tpr <gmx convert-tpr>`.
 
-    .. TODO #.  How do I complete a crashed simulation?
+    .. todo:: #.  How do I complete a crashed simulation?
+
+       Need gmx-cont-crash doc target.
+
+       .. code-block:: none
+
+           This can be easily achieved using the checkpoint reading
+           :ref:`available <gmx-cont-crash>` in |Gromacs| versions newer than 4.
 
-    .. This can be easily achieved using the checkpoint reading
-       :ref:`available <gmx-cont-crash>` in |Gromacs| versions newer than 4.
+    .. todo:: #.  How can I do a simulation at constant pH?
 
-    .. TODO #.  How can I do a simulation at constant pH?
+       Need gmx-howto-cph doc target.
 
-    .. This is a rather large topic, and you should at least read the short
-       :ref:`Constant pH How-To <gmx-howto-cph>` and all of the literature
-       included there to get an overview over the topic.
+        .. code-block:: none
+
+           This is a rather large topic, and you should at least read the short
+           :ref:`Constant pH How-To <gmx-howto-cph>` and all of the literature
+           included there to get an overview over the topic.
 
 #.  How should I compute a single-point energy?
 
@@ -176,9 +186,13 @@ Parameterization and Force Fields
 Analysis and Visualization
 --------------------------
 
-    .. TODO #.  How do I visualize a trajectory?
+.. todo:: #.  How do I visualize a trajectory?
+
+   gmx-howto-visualize doc target:
+
+   .. code-block:: none
 
-    .. Use one of the number of different programs that can visualize
+       Use one of the number of different programs that can visualize
        coordinate :ref:`files and trajectories <gmx-howto-visualize>`.
 
 #.  Why am I seeing bonds being created when I watch the trajectory?
index 334fd1f1b92bec894170d60dfec61764ff039ac7..d01f83f2f9ced31532170041a16b0e1569af9ad9 100644 (file)
@@ -105,7 +105,7 @@ for 43a1, 43a2, 45a3, 53a5, 53a6 and 54a7. The GROMOS force fields are
 * GROMOS 43a1p - 43a1 modified to contain SEP (phosphoserine), TPO (phosphothreonine),
   and PTR (phosphotyrosine) (all PO42- forms), and SEPH, TPOH, PTRH (PO4H- forms).
 
-.. TODO Add new force fields to the list
+.. todo:: Add new force fields to the list
 
 .. _GROMOS: http://www.igc.ethz.ch/gromos/
 .. _reference manual: gmx-manual-parent-dir_
index f70b08ad1d1d3d1c36aa824416865af2fd6d903d..34c6749ee1ff775ab36a2f341f06d941e2154d76 100644 (file)
@@ -20,7 +20,9 @@ For background on algorithms and implementations, see the
 
 |GMX_SOURCE_DOI_STRING|
 
-.. TODO This is going to require more organization now that
+.. todo::
+
+   This is going to require more organization now that
    we are getting more content available.
 
 .. toctree::
index b9901f7b70b1eb1c670721119422b0759ce157bd..36c2ee104ea6a717433862e7c0377d82d45e3f94 100644 (file)
@@ -2,8 +2,9 @@
    See the "run control" section for a working example of the
    syntax to use when making .mdp entries, with and without detailed
    documentation for values those entries might take. Everything can
-   be cross-referenced, see the examples there. TODO Make more
-   cross-references.
+   be cross-referenced, see the examples there.
+
+.. todo:: Make more cross-references.
 
 Molecular dynamics parameters (.mdp options)
 ============================================
index 3627211b620dbe1888181f1157962adda018a151..a0f739e349af75bd52a25d93c96217b79867c788 100644 (file)
@@ -997,9 +997,11 @@ An additional set of subcounters can offer more fine-grained inspection of perfo
 Subcounters are geared toward developers and have to be enabled during compilation. See
 :doc:`/dev-manual/build-system` for more information.
 
-.. TODO In future patch:
-   - red flags in log files, how to interpret wallcycle output
-   - hints to devs how to extend wallcycles
+..  todo::
+
+    In future patch:
+    - red flags in log files, how to interpret wallcycle output
+    - hints to devs how to extend wallcycles
 
 .. _gmx-mdrun-on-gpu:
 
@@ -1054,7 +1056,7 @@ compatibility (please see the :ref:`section below <gmx-pme-gpu-limitations>`).
 GPU computation of short range nonbonded interactions
 .....................................................
 
-.. TODO make this more elaborate and include figures
+.. todo:: make this more elaborate and include figures
 
 Using the GPU for the short-ranged nonbonded interactions provides
 the majority of the available speed-up compared to run using only the CPU.
@@ -1066,7 +1068,7 @@ this problem and thus reduce the calculation time.
 GPU accelerated calculation of PME
 ..................................
 
-.. TODO again, extend this and add some actual useful information concerning performance etc...
+.. todo:: again, extend this and add some actual useful information concerning performance etc...
 
 |Gromacs| now allows the offloading of the PME calculation
 to the GPU, to further reduce the load on the CPU and improve usage overlap between
@@ -1100,7 +1102,7 @@ Known limitations
 GPU accelerated calculation of bonded interactions (CUDA only)
 ..............................................................
 
-.. TODO again, extend this and add some actual useful information concerning performance etc...
+.. todo:: again, extend this and add some actual useful information concerning performance etc...
 
 |Gromacs| now allows the offloading of the bonded part of the PP
 workload to a CUDA-compatible GPU. This is treated as part of the PP
@@ -1171,9 +1173,9 @@ Performance considerations for GPU tasks
 #) The only way to know for sure what alternative is best for
    your machine is to test and check performance.
 
-.. TODO: we need to be more concrete here, i.e. what machine/software aspects to take into consideration, when will default run mode be using PME-GPU and when will it not, when/how should the user reason about testing different settings than the default.
+.. todo:: we need to be more concrete here, i.e. what machine/software aspects to take into consideration, when will default run mode be using PME-GPU and when will it not, when/how should the user reason about testing different settings than the default.
 
-.. TODO someone who knows about the mixed mode should comment further.
+.. todo:: someone who knows about the mixed mode should comment further.
 
 Reducing overheads in GPU accelerated runs
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
@@ -1224,7 +1226,7 @@ Note that assigning fewer resources to :ref:`gmx mdrun` CPU computation
 involves a tradeoff which may outweigh the benefits of reduced GPU driver overhead,
 in particular without HyperThreading and with few CPU cores.
 
-.. TODO In future patch: any tips not covered above
+.. todo:: In future patch: any tips not covered above
 
 Running the OpenCL version of mdrun
 -----------------------------------
index b47083a4a72028f27c671ad9dca98358492392f8..440e566daff7f943c25e638179834fbf031d7cab 100644 (file)
@@ -85,8 +85,14 @@ the options for obtaining the force field parameters are:
 * search the primary literature for publications for parameters for the
   residue that are consistent with the force field that is being used.
 
-.. TODO Once you have determined the parameters and topology for your residue, see
-   :ref:`adding a residue to a force field <gmx-add-new-residue>` for instructions on how to proceed.
+..  todo:: gmx-add-new-residue doc target
+
+    Need gmx-add-new-residue doc target.
+
+    .. code-block:: none
+
+        Once you have determined the parameters and topology for your residue, see
+        :ref:`adding a residue to a force field <gmx-add-new-residue>` for instructions on how to proceed.
 
 Long bonds and/or missing atoms
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^