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

Add DIST output to the KDIST and NDIST #826

Merged
merged 6 commits into from
Jan 12, 2022

Conversation

RomanIlchenko1308
Copy link
Contributor

Improvement of the mapdl.kdist and mapdl.ndist with the implementation of the DIST output.
To obtain the distance between keypoints and nodes without additional hand calculations.

@RomanIlchenko1308 RomanIlchenko1308 added the DO NOT MERGE Not ready to be merged yet label Jan 11, 2022
@RomanIlchenko1308 RomanIlchenko1308 removed the DO NOT MERGE Not ready to be merged yet label Jan 12, 2022
@RomanIlchenko1308
Copy link
Contributor Author

Hi @germa89 may I ask you to check this PR where I have added full output functionality with write description for NDIST and KDIST. Of course for new features I have added unit testing too.

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 job!

@germa89
Copy link
Collaborator

germa89 commented Jan 12, 2022

Approved. @RomanIlchenko1308 feel free to hit the green Squash and merge button.

@RomanIlchenko1308 RomanIlchenko1308 merged commit 90f3fa5 into main Jan 12, 2022
@RomanIlchenko1308 RomanIlchenko1308 deleted the kdist_ndist_modification branch January 12, 2022 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Get the list output instead of the string with "mapdl.ndist".
3 participants