Fixing vale warning

.github/workflows/ci.yml

@@ -57,8 +57,6 @@ jobs:
   doc-style:
     name: "Documentation style check"
     runs-on: ubuntu-latest
-    #Skipping until 2394 is fixed
-    if:  github.ref == 'Skip for now'
     steps:
       - name: "Ansys documentation style checks"
         uses: ansys/actions/doc-style@v4

doc/.vale.ini

@@ -1,31 +1,34 @@
-# Core settings
-# =============
-
-# Location of our `styles`
-StylesPath = "styles"
-
-# The options are `suggestion`, `warning`, or `error` (defaults to “warning”).
-MinAlertLevel = warning
-
-# By default, `code` and `tt` are ignored.
-IgnoredScopes = code, tt
-
-# By default, `script`, `style`, `pre`, and `figure` are ignored.
-SkippedScopes = script, style, pre, figure
-
-# WordTemplate specifies what Vale will consider to be an individual word.
-WordTemplate = \b(?:%s)\b
-
-# List of Packages to be used for our guidelines
-Packages = Google
-
-# Define the Ansys vocabulary
-Vocab = ANSYS
-
-[*.{md,rst}]
-
-# Apply the following styles
-BasedOnStyles = Vale, Google
-
-# Removing Google-specific rule - Not applicable under some circumstances
-Google.Colons = NO
+# Core settings
+# =============
+
+# Location of our `styles`
+StylesPath = "styles"
+
+# The options are `suggestion`, `warning`, or `error` (defaults to “warning”).
+MinAlertLevel = warning
+
+# By default, `code` and `tt` are ignored.
+IgnoredScopes = code, tt
+
+# By default, `script`, `style`, `pre`, and `figure` are ignored.
+SkippedScopes = script, style, pre, figure
+
+# WordTemplate specifies what Vale will consider to be an individual word.
+WordTemplate = \b(?:%s)\b
+
+# List of Packages to be used for our guidelines
+Packages = Google
+
+# Define the Ansys vocabulary
+Vocab = ANSYS
+
+[*.{md,rst}]
+
+# Apply the following styles
+BasedOnStyles = Vale, Google
+
+# `class` and `func` are ignored
+TokenIgnores = (:.*:`.*`)|(<.*>)
+
+# Removing Google-specific rule - Not applicable under some circumstances
+Google.Colons = NO

doc/source/api/unit_testing.rst

@@ -1,14 +1,14 @@
 .. _ref_unit_testing_contributing:
 
-Unit Testing
+Unit testing
 ============
 
 Unit tests validate the software by testing that the logic
 implemented inside a certain method, class, or module is
 working as expected. They should be as atomic and 
 independent as possible.
 
-Unit testing is highly important. The tests check that code
+Unit testing is highly important. The tests verify that code
 changes are consistent with other parts of the code
 and verify that these changes are implemented properly.
 
@@ -34,7 +34,7 @@
 To show how the coverage works, assume that you have
 this library:
 
 **My awesome library**
 
 
 .. code:: python
@@ -82,12 +82,12 @@
 more useful for debugging the function.
 
 
 Continuous Integration and Continuous Deployment (CI/CD) approach
 -----------------------------------------------------------------
 
 Unit tests and integration tests are part of Continuous Integration (CI). 
 The automation of testing, monitoring, and deployment of newly added
 code allows Continuous Deployment (CD) throughout the application
 lifecycle, providing a comprehensive CI/CD approach.
 
 .. figure:: ../images/cicd.jpg
@@ -119,7 +119,7 @@
 MAPDL, which makes them integration tests. If your test
 requires a running MAPDL instance, you can use the PyMAPDL
 `mapdl <mapdl_fixture_>`_ method in your function signature.
 It will be executed upstream of each test and not within all tests.
 
 .. code:: python
 

doc/source/examples/extended_examples/ex_01-gmsh_example/ex_01-gmsh_example.rst

@@ -11,8 +11,8 @@ open source Python meshing library. For more information, visit the Gmsh website
 
 Description
 -----------
-Gmsh is used to import an external geometry file in STL format. The `pymapdl-reader <legacy_reader_docs_>`_ library
-is then used to import the geometry into PyMAPDL. 
+Gmsh is used to import an external geometry file in STL format. The `PyMAPDL Reader <legacy_reader_docs_>`_
+library is then used to import the geometry into PyMAPDL. 
 
 This example makes use of these files: 
 

doc/source/examples/extended_examples/gui/executable.rst

@@ -160,7 +160,9 @@ The **Preprocessing** tab contains input fields for Poisson's ratio, Young modul
 Add a PyVista plotting frame in the window
 ==========================================
 
-Start by importing the `QtInteractor <https://qtdocs.pyvista.org/api_reference.html#qtinteractor>`_ class from the ``pyvistaqt`` package and the :class:`MapdlTheme <ansys.mapdl.core.theme.MapdlTheme>` class from the ``ansys-mapdl-core`` package:
+Start by importing the `QtInteractor <https://qtdocs.pyvista.org/api_reference.html#qtinteractor>`_
+class from the ``pyvistaqt`` package and the :class:`MapdlTheme <ansys.mapdl.core.theme.MapdlTheme>`
+class from the ``ansys-mapdl-core`` package:
 
 .. code:: python
 

doc/source/examples/extended_examples/index.rst

@@ -11,7 +11,7 @@ with other programs, libraries, and features in development.
 +----------------------------+--------------------------------------------------------------------------------------------+
 | `Cart-Pole Simulation`_    | Demonstrates a reinforcement machine learning example using MAPDL through PyMAPDL.         |
 +----------------------------+--------------------------------------------------------------------------------------------+
-| :ref:`extended_example01`  | Demonstrates the interoperability between PyMAPDL and the `gmsh <gmsh_>`_ meshing library. |
+| :ref:`extended_example01`  | Demonstrates the interoperability between PyMAPDL and the `Gmsh <gmsh_>`_ meshing library. |
 +----------------------------+--------------------------------------------------------------------------------------------+
 | :ref:`python_upf_examples` | Demonstrates the interoperability between user-programmable functions in MAPDL and Python. |
 +----------------------------+--------------------------------------------------------------------------------------------+

doc/source/user_guide/convert.rst

@@ -1,5 +1,6 @@
 Translate scripts
 ===================
+
 The `ansys-mapdl-core <pymapdl_docs_>`_
 library contains a few basic functions to translate existing MAPDL
 scripts into PyMAPDL scripts. Ideally, all math and variable setting
@@ -8,7 +9,7 @@ are less transparent and more difficult to debug.
 
 
 Command-line interface
-~~~~~~~~~~~~~~~~~~~~~~
+----------------------
 
 In PyMAPDL v0.64.0 and later, you use the converter from the command line.
 After you have activated and installed the package as described
@@ -40,7 +41,7 @@ The ``pymapdl_convert_script`` command uses the
 Hence, this command accepts most of this function's arguments.
 
 Usage
------
+~~~~~
 
 You can call this command from the terminal with different
 arguments. Here is an example that converts the ``mapdl.dat``
@@ -90,12 +91,13 @@ function documentation.
 
 Caveats
 ~~~~~~~
+
 These examples only show an automatic translation of a verification:
 file and not optimized code. Should it be necessary to pull
-parameters or arrays from ansys, use the :func:`Mapdl.get_value()
-<ansys.mapdl.core.Mapdl.get_value>` function, which is quite similar to the
-MAPDL :func:`Mapdl.get() <ansys.mapdl.core.Mapdl.get>` 
-command shown here:
+parameters or arrays from ansys, use the 
+:func:`Mapdl.get_value() <ansys.mapdl.core.Mapdl.get_value>` function,
+which is quite similar to the MAPDL
+:func:`Mapdl.get() <ansys.mapdl.core.Mapdl.get>` command shown here:
 
 .. code:: pycon
 
@@ -122,6 +124,7 @@ with:
 
 Script translation
 ~~~~~~~~~~~~~~~~~~
+
 Existing Ansys scripts can be translated using the :func:`convert_script() <ansys.mapdl.core.convert_script>`
 function:
 
@@ -151,15 +154,14 @@ Or, you can convert code in form of strings for later processing using the
 The script conversion functions allow some interesting arguments, which you can see in
 the respective :func:`convert_script() <ansys.mapdl.core.convert_script>`
 and :func:`convert_apdl_block() <ansys.mapdl.core.convert_apdl_block>`
-function documentation. Especially interesting are the ``add_imports``, ``comment_solve``, and
-``print_com`` keyword arguments.
-
-Of particular note in the following examples is how most of the
-commands can be called as a method to the Ansys object rather than
-sending a string as a command. Additionally, take note that some
-commands require the :attr:`Mapdl.non_interactive
-<ansys.mapdl.core.Mapdl.non_interactive>` context manager since some
-commands require and may break the server connection for some
+function documentation. Especially interesting are the ``add_imports``, ``comment_solve``,
+and ``print_com`` keyword arguments.
+
+Of particular note in the following examples is how most of the commands can be called
+as a method to the Ansys object rather than sending a string as a command. Additionally,
+take note that some commands require the
+:attr:`Mapdl.non_interactive <ansys.mapdl.core.Mapdl.non_interactive>`
+context manager since some commands require and may break the server connection for some
 interfaces (such as CORBA) or are invalid (as in gRPC).
 
 Also note that APDL macros that use ``*CREATE`` have been replaced
@@ -169,6 +171,7 @@ should it be necessary to insert a ``breakpoint()`` in the script.
 
 Example: VM1 - statically indeterminate reaction force analysis
 ---------------------------------------------------------------
+
 Ansys MAPDL contains over 200 verification files used for Ansys
 validation and demonstration. These validation files are used here to
 demo the use of the PyMAPDL file translator :func:`convert_script()

doc/source/user_guide/mapdl.rst

@@ -17,8 +17,9 @@ Additionally, MAPDL commands
 containing a ``/`` or ``*`` have had those characters removed, unless
 this causes a conflict with an existing name. Most notable is
 ``/SOLU``, which would conflict with ``SOLU``. Therefore,
-``/SOLU`` is renamed to the :func:`Mapdl.slashsolu()
-<ansys.mapdl.core.Mapdl.slashsolu>` method to differentiate it from ``solu``.
+``/SOLU`` is renamed to the
+:func:`Mapdl.slashsolu() <ansys.mapdl.core.Mapdl.slashsolu>`
+method to differentiate it from ``solu``.
 Out of the 1500 MAPDL commands, about 15 start with ``slash (/)`` and 8
 start with ``star (*)``.
 
@@ -35,9 +36,9 @@ Or, these commands can be called using keyword arguments:
 
     mapdl.esel("s", "type", vmin=1)
 
-None of these restrictions apply to commands run with the :func:`Mapdl.run()
-<ansys.mapdl.core.Mapdl.run>` method. It might be easier to run some of
-these commands, such as ``"/SOLU"``:
+None of these restrictions apply to commands run with the
+:func:`Mapdl.run() <ansys.mapdl.core.Mapdl.run>` method. It might be easier
+to run some of these commands, such as ``"/SOLU"``:
 
 .. code:: python
 
@@ -341,9 +342,9 @@ example:
 Prompts
 ~~~~~~~
 Prompts from MAPDL automatically continued as if MAPDL is in batch
-mode. Commands requiring user input, such as the :func:`Mapdl.vwrite()
-<ansys.mapdl.core.Mapdl.vwrite>` method fail and must be entered in
-non-interactively.
+mode. Commands requiring user input, such as the
+:func:`Mapdl.vwrite() <ansys.mapdl.core.Mapdl.vwrite>` method fail
+and must be entered in non-interactively.
 
 
 APDL command logging
@@ -618,8 +619,8 @@ using the :attr:`Mapdl.chain_commands <ansys.mapdl.core.Mapdl.chain_commands>` a
 
 The execution time using this approach is generally 4 to 10 times faster than running
 each command individually. You can then view the final response of
-the chained commands with the :attr:`Mapdl.last_response
-<ansys.mapdl.core.Mapdl.last_response>` attribute.
+the chained commands with the
+:attr:`Mapdl.last_response <ansys.mapdl.core.Mapdl.last_response>` attribute.
 
 .. note::
    Command chaining is not supported in distributed MAPDL.  To improve

doc/source/user_guide/parameters.rst

@@ -4,10 +4,11 @@
 *********************************
 Setting and retrieving parameters
 *********************************
-APDL parameters can be retrieved from and instance of :class:`Mapdl
-<ansys.mapdl.core.mapdl._MapdlCore>` using the :attr:`Mapdl.parameters
-<ansys.mapdl.core.Mapdl.parameters>`.  For example, if you want to use
-MAPDL's :func:`Mapdl.get() <ansys.mapdl.core.Mapdl.get>` method to
+APDL parameters can be retrieved from and instance of 
+:class:`Mapdl <ansys.mapdl.core.mapdl._MapdlCore>`
+using the :attr:`Mapdl.parameters <ansys.mapdl.core.Mapdl.parameters>`.
+For example, if you want to use MAPDL's
+:func:`Mapdl.get() <ansys.mapdl.core.Mapdl.get>` method to
 populate a parameter, you can then access the parameter with code:
 
 .. code:: pycon

doc/source/user_guide/plotting.rst

@@ -52,9 +52,9 @@ You plot lines within Python using the :func:`Mapdl.lplot() <ansys.mapdl.core.Ma
 Area and volume plotting
 ~~~~~~~~~~~~~~~~~~~~~~~~
 You can using Boolean operations to obtain more complex geometry and
-visualize them using the :func:`Mapdl.vplot()
-<ansys.mapdl.core.Mapdl.vplot>` method. This example cuts the initial
-area with the eight circles and then extrudes it.
+visualize them using the :func:`Mapdl.vplot() <ansys.mapdl.core.Mapdl.vplot>`
+method. This example cuts the initial area with the eight circles and then
+extrudes it.
 
 .. code:: pycon
 
@@ -202,8 +202,8 @@ method:
 
 .. note::
     Because boundary conditions can only target nodes, you can
-    only use ``plot_bc`` as an argument in the :func:`Mapdl.nplot()
-    <ansys.mapdl.core.Mapdl.nplot>` method.
+    only use ``plot_bc`` as an argument in the
+    :func:`Mapdl.nplot() <ansys.mapdl.core.Mapdl.nplot>` method.
 
 
 

doc/source/user_guide/pool.rst

@@ -2,12 +2,11 @@
 
 Create a pool of MAPDL instances
 ================================
-PyMAPDL contains the :class:`LocalMapdlPool
-<ansys.mapdl.core.LocalMapdlPool>` class to simplify creating multiple
-local instances of the :class:`Mapdl <ansys.mapdl.core.mapdl._MapdlCore>`
-class for batch processing. This can be used for the batch processing of a
-set of input files, convergence analysis, or other batch related
-processes.
+PyMAPDL contains the :class:`LocalMapdlPool <ansys.mapdl.core.LocalMapdlPool>`
+class to simplify creating multiple local instances of the
+:class:`Mapdl <ansys.mapdl.core.mapdl._MapdlCore>` class for batch processing.
+This can be used for the batch processing of a set of input files,
+convergence analysis, or other batch related processes.
 
 This code creates a pool: