Documentation#2017
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates documentation for the Gazebo simulation setup with OM1, streamlining the installation workflow and correcting various documentation issues.
Changes:
- Removed Step 7 (Start OM1) from the Gazebo setup, consolidating the workflow
- Updated dependency installation instructions to use
python3-devinstead of deprecatedpython-all-dev/python-dev - Added virtual environment setup and PYTHONPATH configuration for unitree-sdk
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| mintlify/examples/gazebo.mdx | Streamlined Gazebo setup by removing OM1 startup step and renumbering subsequent steps; added venv and PYTHONPATH setup |
| docs/examples/gazebo.mdx | Identical changes to mintlify version for consistency |
| mintlify/developing/6_actions.mdx | Fixed tree structure formatting and added missing orchestrator entry |
| docs/developing/6_actions.mdx | Identical changes to mintlify version for consistency |
| mintlify/developing/1_get-started.mdx | Updated Linux dependency note to specify python3-dev |
| docs/developing/1_get-started.mdx | Identical changes to mintlify version for consistency |
| mintlify/developer_cookbook/introduction.mdx | Capitalized "Input" and added new Action Plugin entry to capabilities list |
| docs/developer_cookbook/introduction.mdx | Capitalized "Input" for consistency |
| config/gazebo_sim.json5 | New configuration file for Gazebo simulation with mode definitions |
| README.md | Updated apt-get command to use python3-dev instead of python-dev |
| .github/ISSUE_TEMPLATE/🌟-bounty.md | Corrected GitHub organization name from OpenmindAGI to Openmind |
Comments suppressed due to low confidence (1)
docs/examples/gazebo.mdx:1
- The hardcoded Python version 3.10 in the path may cause issues if users have a different Python version installed. Consider using a version-agnostic approach or document the Python version requirement.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Export the python path to your virtual environment | ||
|
|
||
| ```bash | ||
| export PYTHONPATH=$PYTHONPATH:$(pwd)/.venv/lib/python3.10/site-packages |
There was a problem hiding this comment.
The hardcoded Python version 3.10 in the path may cause issues if users have a different Python version installed. Consider using a version-agnostic approach or document the Python version requirement.
| export PYTHONPATH=$PYTHONPATH:$(pwd)/.venv/lib/python3.10/site-packages | |
| export PYTHONPATH="$PYTHONPATH:$(python -c 'import site; print(site.getsitepackages()[0])')" |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Overview
This PR modifies gazebo setup and dependency requirements for OM1.
It also adds a new config file to run with Gazebo simulation