-
Notifications
You must be signed in to change notification settings - Fork 128
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.
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
feat: database
submodule
#3748
base: feat/main_commands
Are you sure you want to change the base?
feat: database
submodule
#3748
Conversation
Thanks for opening a Pull Request. If you want to perform a review write a comment saying: @ansys-reviewer-bot review |
…mapdl into feat/database_submodule
|
||
APDL Command: CMDELE | ||
.. _CMLIST_notes: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe that the references should be located above the section header. So you can reference to them using :ref:
.
See: https://www.sphinx-doc.org/en/master/usage/referencing.html#role-ref
and for each component list the underlying entity ID's. :ref:`cmlist`, ``Name`` - List the specified | ||
component. :ref:`cmlist`, ``Name``,EXPA - List specified component along with all underlying entity | ||
ID's. :ref:`cmlist`, EXPA, ``Entity`` - List all selected components of specified entity | ||
type. For each component also list the underlying entity ID's. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this format is wrong...
0 - Component is not updated during remeshing and therefore contains only initial mesh elements | ||
(default). 1 - Component is updated during remeshing to contain the updated elements. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Format is off
:ref:`cmplot`, ``CNAME`` - Plots the specified component (if selected). :ref:`cmplot`, ``CNAME``, | ||
ALL - Plot component in the context of all other selected entity components of the same type as the | ||
component. :ref:`cmplot` - Plot the first eleven selected components. :ref:`cmplot`,ALL - Plot all | ||
selected components. :ref:`cmplot`,N or :ref:`cmplot`,P - Plot next or previous set of eleven | ||
components. :ref:`cmplot`,ALL, ``Entity`` - Plot all selected components of type specified in | ||
``Entity``. :ref:`cmplot`, ``Entity`` - Plot components of type specified in ``Entity``, from the | ||
first eleven components. :ref:`cmplot`,N, ``Entity`` - Plot components of type specified in | ||
``Entity``, if any, from the next set of eleven components (substitute P for N to plot from previous | ||
set). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Format?
edit this | ||
set of commands. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
weird break
(rotations); ROT (ROTX, | ||
ROTY, and ROTZ); DISP (U and ROT); HDSP (Hydrostatic pressure). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
weird break line
Used only with ``Type`` = S, A, or U. Valid lables are: | ||
|
||
**Structural labels** : UX, UY, or UZ (displacements); U (UX, UY, and UZ) ; ROTX, ROTY, or ROTZ | ||
(rotations); ROT (ROTX, | ||
ROTY, and ROTZ); DISP (U and ROT); HDSP (Hydrostatic pressure). | ||
**Thermal labels** : TEMP, TBOT, TE2, TE3...., TTOP (temperature). | ||
**Acoustic labels** : PRES (pressure); UX, UY, or UZ (displacements for FSI coupled elements). | ||
**Electric labels** : VOLT (voltage); EMF (electromotive force drop); CURR (current). | ||
**Magnetic labels** : MAG (scalar magnetic potential); AZ (vector magnetic potential); A (AZ); CURR | ||
(current). | ||
**Structural force labels** : FX, FY, or FZ (forces); F (FX, FY, and FZ); MX, MY, or MZ (moments); M | ||
(MX, MY, and MZ); FORC (F | ||
and M); DVOL (fluid mass flow rate). | ||
**Thermal force labels** : HEAT, HBOT, HE2, HE3...., HTOP (heat flow). | ||
**Fluid flow force label** : FLOW (fluid flow). | ||
**Electric force labels** : AMPS (current flow); CHRG (electric charge). | ||
**Magnetic force labels** : FLUX (scalar magnetic flux); CSGZ (magnetic current segment). | ||
**Diffusion labels** : CONC (concentration); RATE (diffusion flow rate). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are repeating the same content as dof1
... what did we agree about this? To repeat or mention same as argX
?
If ``VMIN`` = ``VMAX``, ``Toler`` = 0.005 x ``VMIN``. | ||
If ``VMIN`` = ``VMAX`` = 0.0, ``Toler`` = 1.0E-6. | ||
If ``VMAX`` ≠ ``VMIN``, ``Toler`` = 1.0E-8 x ( ``VMAX`` - ``VMIN`` ). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it might be a good idea to make this bullet points, or similar. Otherwise, sphinx will render it as just one line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work over all. Some minor comments/fix are needed.
If ``VMIN`` = ``VMAX``, ``Toler`` = 0.005 x ``VMIN``. | ||
If ``VMIN`` = ``VMAX`` = 0.0, ``Toler`` = 1.0E-6. | ||
If ``VMAX`` ≠ ``VMIN``, ``Toler`` = 1.0E-8 x ( ``VMAX`` - ``VMIN`` ). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same, better a bullet list.
item : str | ||
The description of the argument is missing in the Python function. Please, refer to the `command | ||
documentation | ||
<https://ansyshelp.ansys.com/Views/Secured/corp/v232/en//ans_cmd/Hlp_C_ESEL.html>`_ for further | ||
information. | ||
|
||
Item | ||
Label identifying data. Valid item labels are shown in the table below. Some items also require a | ||
component label. If Item = PICK (or simply "P"), graphical picking is enabled and all remaining | ||
command fields are ignored (valid only in the GUI). Defaults to KP. | ||
comp : str | ||
The description of the argument is missing in the Python function. Please, refer to the `command | ||
documentation | ||
<https://ansyshelp.ansys.com/Views/Secured/corp/v232/en//ans_cmd/Hlp_C_ESEL.html>`_ for further | ||
information. | ||
|
||
Comp | ||
Component of the item (if required). Valid component labels are shown in the table below. | ||
vmin : str | ||
The description of the argument is missing in the Python function. Please, refer to the `command | ||
documentation | ||
<https://ansyshelp.ansys.com/Views/Secured/corp/v232/en//ans_cmd/Hlp_C_ESEL.html>`_ for further | ||
information. | ||
|
||
VMIN | ||
Minimum value of item range. Ranges are keypoint numbers, coordinate values, attribute numbers, | ||
etc., as appropriate for the item. A component name (as specified on the CM (p. 338) command) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The description of the argument is missing in the Python function. Please, refer to the `command documentation
??
- Element number. | ||
* - ADJ | ||
- | ||
- Elements adjacent to element ``VMIN`` ( ``VMAX`` and ``VINC`` fields are ignored). Only elements (of the same dimensionality) adjacent to lateral faces are considered. Progression continues until edge of model or until more than two elements are adjacent at a face. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to break the line??
The description of the argument is missing in the Python function. Please, refer to the `command | ||
documentation | ||
<https://ansyshelp.ansys.com/Views/Secured/corp/v232/en//ans_cmd/Hlp_C_NSEL.html>`_ for further | ||
information. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again missing the description of the argument... ??
The description of the argument is missing in the Python function. Please, refer to the `command | ||
documentation | ||
<https://ansyshelp.ansys.com/Views/Secured/corp/v232/en//ans_cmd/Hlp_C_VSEL.html>`_ for further | ||
information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
again...
item : str | ||
The description of the argument is missing in the Python function. Please, refer to the `command | ||
documentation | ||
<https://ansyshelp.ansys.com/Views/Secured/corp/v232/en//ans_cmd/Hlp_C_ASEL.html>`_ for further | ||
information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same...
The description of the argument is missing in the Python function. Please, refer to the `command | ||
documentation | ||
<https://ansyshelp.ansys.com/Views/Secured/corp/v232/en//ans_cmd/Hlp_C_UNITS.html>`_ for further | ||
information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
??
PREP7 topic commands (and their corresponding topics) are: | ||
|
||
InformalTables need to be added. | ||
SOLUTION topic commands (and their corresponding topics) are: | ||
|
||
InformalTables need to be added. | ||
POST1 topic commands (and their corresponding topics) are: | ||
|
||
InformalTables need to be added. | ||
POST26 topic commands (and their corresponding topics) are: | ||
|
||
InformalTables need to be added. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope you took note of this TODOs
The towncrier template has been successfully rewritten with the default template. |
Description
This PR follows the PyConverter-XML2Py integration plan to automate the PyMAPDL_commands documentation.
The changes have been generated using pyconverter-xml2py and more specifically mapdl-cmd-conv.
This PR focus on the
database
submodule.Pinging @ansys/pymapdl-developers for visibility. Feel free to provide any feedback on the way the docstrings and the source code generation are handled.
Issue linked
This PR is meant to be merged within the feat/main_commands branch. The latter will gather all the submodule changes, one by one, prior to be merged to the main branch.
Checklist
draft
if it is not ready to be reviewed yet.feat: adding new MAPDL command
)