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

Exposed all existing documentation in Articulated_agents, articulated_agent_controllers, core, and config modules #1736

Merged
merged 9 commits into from
Jan 17, 2024

Conversation

henrysamer
Copy link
Contributor

Motivation and Context

Improve documentation coverage and quality in habitat-lab. Add type hints to functions and classes to enhance clarity, facilitate static analysis, and assist in bug detection.

How Has This Been Tested

Changes tested through local build verification and validation using tools like mypy.

Types of changes

  • [Docs change] Addition or changes to the documentation
  • [Bug Fix] (non-breaking change which fixes an issue)

Checklist

  • My code follows the code style of this project.
  • [ X ] I have updated the documentation if required.
  • I have read the CONTRIBUTING document.
  • I have completed my CLA (see CONTRIBUTING)
  • I have added tests to cover my changes if required.

@henrysamer henrysamer requested review from mosra and aclegg3 January 5, 2024 00:58
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jan 5, 2024
resets to 0.
:property ee_offset: The 3D offset from the end-effector link to the true
end-effector position.
:property arm_init_params: The starting joint angles of the arm. If None, resets to 0.
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrapping the docstring like it was before shouldn't cause any problems for the doc generator. The successive lines just need to be indented respective to the first :property : line.

@@ -29,7 +30,7 @@ class Motion:
:param joints_quat_array: num_poses x num_joints x 4 array, containing the join orientations
:param transform_array: num_poses x 4 x 4 array, containing the root transform
:param displacement: on each pose, how much forward displacement was there?
Used to measure how many poses we should advance to move a cerain amount
Used to measure how many poses we should advance to move a cerain amount
Copy link
Contributor

Choose a reason for hiding this comment

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

I.e., this will most probably cause a problem -- it should be indented like before, or on a single line.

@henrysamer henrysamer changed the title Update completed class/function documentation in labdocsv1 Exposed all existing documentation in Articulated_agents, articulated_agent_controllers, core, and config modules Jan 16, 2024
@henrysamer henrysamer marked this pull request as ready for review January 17, 2024 18:15
Copy link
Contributor

@aclegg3 aclegg3 left a comment

Choose a reason for hiding this comment

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

One typo, otherwise good to go.

:param joints_quat: list or array of num_joints * 4 elements, with the rotation quaternions
:param root_transform: Matrix4 with the root transform.
:param root_transform: Matrix4 with the root trasnform.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
:param root_transform: Matrix4 with the root trasnform.
:param root_transform: Matrix4 with the root transform.

@henrysamer henrysamer merged commit 727b100 into main Jan 17, 2024
4 checks passed
@henrysamer henrysamer deleted the labdocsv1 branch January 18, 2024 14:20
dannymcy pushed a commit to dannymcy/habitat-lab that referenced this pull request Jul 8, 2024
…_agent_controllers, core, and config modules (facebookresearch#1736)

* Update completed class/function documentation in labdocsv1

* pre-commit passed

* Exposed class modules

* Exposed methods in Articulated_agents, articulated_agent_controllers, core, and config modules

* fix duplicate param causing doc build failure

---------

Co-authored-by: “Sam <“henrysamer@meta.com”>
Co-authored-by: aclegg3 <alexanderwclegg@gmail.com>
HHYHRHY pushed a commit to SgtVincent/EMOS that referenced this pull request Aug 31, 2024
…_agent_controllers, core, and config modules (facebookresearch#1736)

* Update completed class/function documentation in labdocsv1

* pre-commit passed

* Exposed class modules

* Exposed methods in Articulated_agents, articulated_agent_controllers, core, and config modules

* fix duplicate param causing doc build failure

---------

Co-authored-by: “Sam <“henrysamer@meta.com”>
Co-authored-by: aclegg3 <alexanderwclegg@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants