gazebosim / gazebo_test_cases

4 stars 1 forks source link

docs: install_ubuntu_src #1699

Closed azeey-test closed 1 week ago

azeey-test commented 2 weeks ago

Configuration

Links

Process

Step 1 - Instructions work

Step 2 - Images (if there are any) match the result

Mechazo11 commented 2 weeks ago

Interested

Mechazo11 commented 1 week ago
j-rivero commented 1 week ago

Great review @Mechazo11 ! I'm processing your suggestions.

Commands pip install vcstool || pip3 install vcstool and pip install -U colcon-common-extensions || pip3 install -U colcon-common-extensions throws externally-managed-enviornment error. This appears to be a reported problem since Python 3.11+.

https://github.com/gazebosim/docs/pull/493

j-rivero commented 1 week ago

CMake Error: Parse error in command line argument: QT_QML_DEBUG Should be: VAR:type=value

Fixed by https://github.com/gazebosim/docs/pull/494, first commit

sudo apt-get install git may be unnecessary.

Used by vcs under the hood, quite sure we need it.

  • Adding instructions for adding 8GB - 16GB additional swap space. When building multiple packages such as gz-gui9, gz-physics8, gz-sensors9, the ram requirement did exceed 15GB.

Added a note about the number of RAM needed and how to limit the compilation threads. https://github.com/gazebosim/docs/pull/494, second commit.

  • For the Troubleshooting section there should be a TOC or short one-two linear paragraph stating the type of issues and solutions currently covered in that document file.

Leave this one for you if you are interested in patching the doc :)

Mechazo11 commented 1 week ago

@azeey-test I think this one can be closed

j-rivero commented 1 week ago

@azeey-test I think this one can be closed

as said, great work!