Skip to content

Conversation

@ajain-work
Copy link
Collaborator

@ajain-work ajain-work commented Jun 16, 2022

Doc for visualization
doc
n

super().add_directive_header(sig)
self.add_line(" ", self.get_sourcename())

def add_content(
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Only this method is modified.

@ajain-work ajain-work force-pushed the feat/visualization_doc branch from 1257ce7 to ee38cea Compare June 16, 2022 08:08
@@ -0,0 +1,186 @@
from enum import IntEnum
from typing import Any, Optional
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

# Bottom border
self.add_line(f'{"="*key_max}{" "*col_gap}{"="*val_max}', source_name)
self.add_line("", source_name)
self.add_line("", source_name)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Used Varun's code for settings API.

@ajain-work ajain-work force-pushed the feat/visualization_doc branch from 906143f to 3589493 Compare June 16, 2022 09:33

.. currentmodule:: {{ module }}

.. autoclass:: {{ objname }}
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 if you include the :members: and :inherited-members: directives here, it will create the method docs.

data_dicts[obj_name]["parent"] = parent_name
data_dicts[obj_name]["obj"] = clss
data_dicts[obj_name]["attr"] = {"Member": "Summary"}
data_dicts[obj_name]["cmd"] = {"command": "Summary"}
Copy link
Collaborator

@seanpearsonuk seanpearsonuk Jun 24, 2022

Choose a reason for hiding this comment

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

command starts lower-case which is different to the others.

@ajain-work ajain-work force-pushed the feat/visualization_doc branch from 65232e7 to 1f8387f Compare July 3, 2022 08:04
@ajain-work ajain-work changed the base branch from main to feat/visualization_doc_string July 3, 2022 08:05
@ajain-work ajain-work force-pushed the feat/visualization_doc branch from f8ac31b to fc28950 Compare July 4, 2022 03:02
@ajain-work ajain-work marked this pull request as ready for review July 4, 2022 03:07
@ajain-work ajain-work force-pushed the feat/visualization_doc branch from 9e2b1b3 to dec2847 Compare July 5, 2022 07:59
@ajain-work ajain-work merged commit a3585dd into feat/visualization_doc_string Jul 5, 2022
@ajain-work ajain-work deleted the feat/visualization_doc branch July 5, 2022 08:33
ajain-work added a commit that referenced this pull request Jul 5, 2022
* Doc strings.

* Visualization Doc. (#29)

* Visualization doc

* Example updated. (#49)
ajain-work added a commit that referenced this pull request Jul 5, 2022
* Doc strings.

* Visualization Doc. (#29)

* Visualization doc

* Example updated. (#49)
@ajain-work ajain-work mentioned this pull request Jul 5, 2022
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.

4 participants