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.

Sign up for GitHub

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

Jump to bottom

Add docs to command output #822

Merged
merged 111 commits into from
Jan 13, 2022
Merged

Add docs to command output #822

merged 111 commits into from
Jan 13, 2022

Conversation

germa89
Copy link
Collaborator

@germa89 germa89 commented Jan 11, 2022

Adding documentation to rich command output

Close #791

germa89 added 30 commits December 20, 2021 13:59
@germa89
Changed signal termination
2ef7767
@germa89
Adding CommandOutput class.
5f84f1c
@germa89
Substituting the output in mapdl_grpc for custom class.
9dc57fc
@germa89
Merge branch 'fix/os-error-when-using-os.kill-in-unit-testing' into f…
2c508bb 
…eat/richer-command-output
@germa89
Making sure all the methods call the parent class (str)
28a838f
@germa89
Testing simplied version
18525a7
@germa89
Added unit tests
9c2b52f
@germa89
Added unit tests
915444c
@germa89
Fixing the style
652ca3d
@germa89
Merge branch 'feat/richer-command-output' of https://github.com/pyans…
c8ec465 
…ys/pymapdl into feat/richer-command-output
@germa89
Fixing grammar.
9cbda1e
@germa89
Fixing grammar.
52c3e4a
@germa89
Merge branch 'feat/richer-command-output' of https://github.com/pyans…
1ff363e 
…ys/pymapdl into feat/richer-command-output
@germa89
Merge branch 'feat/richer-command-output' of https://github.com/pyans…
12b143f 
…ys/pymapdl into feat/richer-command-output
@germa89
Add test_class unit
8bc2768
@germa89
Using first implementation because the second fail because of the mod…
64dbd92 
…ified `__getattribute_`
@germa89
Changing implementation to not overwrite __class__.
c110b88
@germa89
Fixing sphinx building by rewriting __class__ method to not be overwr…
9fdc7f7 
…iten.
@germa89
Style check.
53aab29
@germa89
changing the API, cmd=command, and command= full command (cmd + args)
d98bd79
@germa89
Adding CommandOutputDataframe class.
edde78f
@germa89
Small changes.
3f17960
@germa89
Big restructure.
71665ec 
Now we detect the underlying data array based on a predefined magic word (start of data) and an empty line (end of data).
@germa89
Added check if output is modified by /verify
bcfd4b8
@germa89
Added docstrings to classes.
bdb0ae3
@germa89
Moved fixtures to main conf file
d7a15d2
@germa89
Changed class name, added supported commands.
2bbe92d 
Added support to Elist and Nlist
@germa89
Added test units.
3aece4b
@germa89
Added more unit test.
252660a
@germa89
Fixing style.
9449878
@germa89 germa89 marked this pull request as ready for review January 11, 2022 17:21
akaszynski
akaszynski reviewed Jan 12, 2022
View reviewed changes
@germa89 germa89 changed the title Docs/command output doc Add docs to command output Jan 12, 2022
akaszynski
akaszynski approved these changes Jan 13, 2022
View reviewed changes
Copy link
Collaborator

@akaszynski akaszynski left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for cleaning up the docs.

@akaszynski akaszynski merged commit c4a3a36 into main Jan 13, 2022
@akaszynski akaszynski deleted the docs/command-output-doc branch January 13, 2022 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akaszynski akaszynski akaszynski approved these changes

Assignees

@germa89 germa89

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@germa89 @akaszynski