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: aux2 submodule #3683

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

feat: aux2 submodule #3683

wants to merge 10 commits into from

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

Note
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 added documentation Documentation related (improving, adding, etc) enhancement Improve any current implemented feature labels Jan 21, 2025
@clatapie clatapie self-assigned this Jan 21, 2025
@clatapie clatapie requested a review from a team as a code owner January 21, 2025 11:10
@clatapie clatapie requested review from germa89 and pyansys-ci-bot and removed request for a team January 21, 2025 11:10
@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

@clatapie clatapie changed the title feat: aux1 submodule feat: aux2 submodule Jan 21, 2025
@github-actions github-actions bot added the new feature Request or proposal for a new feature label Jan 21, 2025
@github-actions github-actions bot added CI/CD Related with CICD, Github Actions, etc maintenance General maintenance of the repo (libraries, cicd, etc) labels Jan 22, 2025
@github-actions github-actions bot removed CI/CD Related with CICD, Github Actions, etc maintenance General maintenance of the repo (libraries, cicd, etc) labels Jan 22, 2025
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.

Very minor comments which you can leave aside for future iterations.

Otherwise, looks good to me!

mass matrix values (and possibly element damping matrix values) are incorporated into the globally
assembled stiffness matrix. Thus, the globally assembled stiffness matrix represents more than the
stiffness of the model for these analysis types. Please refer to the `Theory Reference
<https://ansyshelp.ansys.com/Views/Secured/corp/v232/en/ans_thry/thy_biblio.html>`_ Mechanical APDL
Copy link
Collaborator

Choose a reason for hiding this comment

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

Parsing error?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am not following, what are you referring to here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I got it now, sorry for the delay 😄
I am investigating!

Comment on lines +125 to +126
files are
stored by row and not column, when the :ref:`hbmat` command is used with a non-symmetric matrix, the
Copy link
Collaborator

Choose a reason for hiding this comment

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

Strange new line...

Comment on lines +295 to +297
stiffness of the model for these analysis types. Please refer to the `Theory Reference
<https://ansyshelp.ansys.com/Views/Secured/corp/v232/en/ans_thry/thy_biblio.html>`_ Mechanical APDL
Theory Reference for more details.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same duplicated Theory Reference

@germa89
Copy link
Collaborator

germa89 commented Jan 30, 2025

Once you have take care of my comments, you can move back the files to their previous locations. Since my comments should not change the code, rather the implementation, you could do git revert.

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) enhancement Improve any current implemented feature new feature Request or proposal for a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants