Skip to content

docs(providers): add comprehensive __init__ parameter documentation for OdomProvider#1644

Merged
openminddev merged 2 commits intoOpenMind:mainfrom
shajiang919:docs/odom-provider-init-docs
Jan 18, 2026
Merged

docs(providers): add comprehensive __init__ parameter documentation for OdomProvider#1644
openminddev merged 2 commits intoOpenMind:mainfrom
shajiang919:docs/odom-provider-init-docs

Conversation

@shajiang919
Copy link
Copy Markdown
Contributor

added missing parameter docs for odomprovider init method.

details:

  • added parameters section for URID, use_zenoh, channel
  • documented zenoh vs cyclonedds communication methods
  • added notes on singleton pattern and initialization flow
  • explained multiprocessing/threading architecture

replaces single-line docstring with complete numpy-style documentation.

@shajiang919 shajiang919 requested review from a team as code owners January 18, 2026 15:59
@github-actions github-actions Bot added robotics Robotics code changes python Python code labels Jan 18, 2026
Refactored the OdomProvider class docstring to provide clearer and more concise parameter descriptions for URID, use_zenoh, and channel. Removed redundant notes and implementation details to improve readability.
@openminddev openminddev merged commit 8e7b0f5 into OpenMind:main Jan 18, 2026
1 check passed
aydnOktay pushed a commit to aydnOktay/OM1 that referenced this pull request Jan 19, 2026
…or OdomProvider (OpenMind#1644)

* docs(providers): add comprehensive __init__ parameter documentation for OdomProvider

* Simplify and clarify OdomProvider docstring parameters

Refactored the OdomProvider class docstring to provide clearer and more concise parameter descriptions for URID, use_zenoh, and channel. Removed redundant notes and implementation details to improve readability.

---------

Co-authored-by: openmindev <boyuan.eth@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Python code robotics Robotics code changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants