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.

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

Open
wants to merge 16 commits into
base: feat/main_commands
Choose a base branch
from
Open

Conversation

clatapie
Copy link
Contributor

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

@clatapie clatapie self-assigned this Feb 17, 2025
@clatapie clatapie requested a review from a team as a code owner February 17, 2025 17:28
@clatapie clatapie requested review from germa89 and pyansys-ci-bot and removed request for a team February 17, 2025 17:28
@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@github-actions github-actions bot added documentation Documentation related (improving, adding, etc) new feature Request or proposal for a new feature labels Feb 17, 2025
@clatapie clatapie changed the title feat: adding database submodule feat: database submodule Feb 17, 2025

APDL Command: CMDELE
.. _CMLIST_notes:
Copy link
Collaborator

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

Comment on lines 70 to 73
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.
Copy link
Collaborator

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...

Comment on lines +209 to +210
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.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Format is off

Comment on lines +435 to +443
: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).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Format?

Comment on lines +94 to +95
edit this
set of commands.
Copy link
Collaborator

Choose a reason for hiding this comment

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

weird break

Comment on lines 66 to 67
(rotations); ROT (ROTX,
ROTY, and ROTZ); DISP (U and ROT); HDSP (Hydrostatic pressure).
Copy link
Collaborator

Choose a reason for hiding this comment

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

weird break line

Comment on lines 83 to 100
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).
Copy link
Collaborator

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?

Comment on lines 348 to 350
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`` ).
Copy link
Collaborator

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.

Copy link
Collaborator

@germa89 germa89 left a 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.

Comment on lines +552 to +554
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`` ).
Copy link
Collaborator

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.

Comment on lines 806 to 808
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)
Copy link
Collaborator

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.
Copy link
Collaborator

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??

Comment on lines +1141 to 1145
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.

Copy link
Collaborator

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... ??

Comment on lines 1603 to 1606
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.
Copy link
Collaborator

Choose a reason for hiding this comment

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

again...

Comment on lines +1890 to +1894
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.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same...

Comment on lines +211 to +214
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.
Copy link
Collaborator

Choose a reason for hiding this comment

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

??

Comment on lines +453 to +464
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.
Copy link
Collaborator

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

Copy link
Contributor

github-actions bot commented Mar 5, 2025

The towncrier template has been successfully rewritten with the default template.
Please ensure that sphinx-design is included as a dependency in your project's documentation requirements.
Additionally, add sphinx-design to the extensions list in your conf.py file.
For more information, please refer to the migration guide at:
https://actions.docs.ansys.com/version/stable/migrations/index.html#migration-guide

@github-actions github-actions bot added CI/CD Related with CICD, Github Actions, etc dependencies maintenance General maintenance of the repo (libraries, cicd, etc) labels Mar 7, 2025
@github-actions github-actions bot removed CI/CD Related with CICD, Github Actions, etc dependencies maintenance General maintenance of the repo (libraries, cicd, etc) labels Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation related (improving, adding, etc) new feature Request or proposal for a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants