moveit2 icon indicating copy to clipboard operation
moveit2 copied to clipboard

Improve docstrings in moveit_py '.pyi' stubs for better usability

Open VibhuSharma19 opened this issue 8 months ago • 2 comments

docs(moveit_py): enhance .pyi stub files with detailed doc-strings

This commit improves the type stub files (.pyi) within the moveit_py package by adding comprehensive doc-strings for all public classes, methods, and properties. The enhancements include:

  • Usage descriptions for classes and interfaces
  • Argument details with types and purpose
  • Return value descriptions
  • Improved clarity for users relying on static analysis tools (e.g. mypy, Pyright)
  • Better integration with editor IntelliSense and autocomplete features

These changes aim to improve developer experience, maintainability, and usability of the Python interface of MoveIt. No changes were made to runtime behavior.

Signed-off-by: Vibhu Sharma [email protected]

Description

This pull request enhances the type stub files (.pyi) in the moveit_py package by adding comprehensive and standardized doc-strings for all public-facing classes, methods, and properties.

These improvements provide detailed information such as:

  • Purpose and usage of classes/interfaces
    
  • Argument names, types, and descriptions
    
  • Return value documentation
    
  • Semantic clarity for developers and contributors
    

Motivation

This change is intended to:

  • Enable rich IntelliSense/autocomplete in editors like VSCode 
    
  • Serve as inline documentation for better onboarding and usage understanding
    
  • Ensure the moveit_py Python interface is easier to use and maintain
    

Runtime Impact

No changes were made to runtime behavior.
This is a purely documentation-oriented update confined to .pyi files.

Checklist

  • [ ] Required by CI: Code is auto formatted using clang-format
  • [ ] Extend the tutorials / documentation reference
  • [ ] Document API changes relevant to the user in the MIGRATION.md notes
  • [ ] Create tests, which fail without this PR reference
  • [ ] Include a screenshot if changing a GUI
  • [ ] While waiting for someone to review your request, please help review another open pull request to support the maintainers

VibhuSharma19 avatar Apr 07 '25 17:04 VibhuSharma19

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 45.84%. Comparing base (89acadd) to head (1285d0a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3424      +/-   ##
==========================================
- Coverage   45.86%   45.84%   -0.01%     
==========================================
  Files         717      717              
  Lines       62628    62628              
  Branches     7580     7580              
==========================================
- Hits        28715    28705      -10     
- Misses      33748    33756       +8     
- Partials      165      167       +2     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Apr 08 '25 00:04 codecov[bot]

Hi, Kindly review and merge the changes at your earliest convenience. Let me know if any updates are needed. Thanks!

VibhuSharma19 avatar May 09 '25 11:05 VibhuSharma19

This PR is stale because it has been open for 45 days with no activity. Please tag a maintainer for help on completing this PR, or close it if you think it has become obsolete.

github-actions[bot] avatar Jun 30 '25 12:06 github-actions[bot]

This pull request is in conflict. Could you fix it @VibhuSharma19?

mergify[bot] avatar Jun 30 '25 12:06 mergify[bot]

This PR is stale because it has been open for 45 days with no activity. Please tag a maintainer for help on completing this PR, or close it if you think it has become obsolete.

github-actions[bot] avatar Aug 20 '25 12:08 github-actions[bot]