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

Initial clean-up of VCS repos and README files #37

Open
wants to merge 10 commits into
base: humble-devel
Choose a base branch
from

Conversation

Kotochleb
Copy link
Contributor

No description provided.

@Kotochleb Kotochleb marked this pull request as draft February 6, 2025 18:15
README.md Show resolved Hide resolved
> [!CAUTION]
> Current demos only make use of `agimus_dev_container:humble-devel-control`. There is no need to pull Docker images with vision dependencies.

Docker image is provided in a form of Development Container and can be found at [agimus-project/agimus_dev_container](https://gitlab.laas.fr/agimus-project/agimus_dev_container). Use branch `humble-devel` and refer to the README.md file for more information on how to use it.
Copy link
Contributor

Choose a reason for hiding this comment

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

should we just point to readme maybe? Also, in readme info gets a bit lost I think (f.e. setup.sh), so we should either say FOLLOW VERY CAREFULLY the README. Or just copy a sequence of steps in short form (like I did below) and maybe send for "if any questions look on readme"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be better now

@Kotochleb Kotochleb marked this pull request as ready for review February 7, 2025 13:17
ros2 launch agimus_demo_<demo-name> bringup.launch.py robot_ip:=<robot-ip> use_rviz:=true
```

## Running demos with Docker
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
## Running demos with Docker
## Option 1: Running demos with Docker


Now you are ready to run all of your demos!

## Building dependencies from source
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
## Building dependencies from source
## Option 2: Building dependencies from source

Follow [these steps](doc/docker.md) to set up the docker for the project.
Agimus Demos is a set of ROS 2 packages used to launch different demos developed for [Agimus Project](https://www.agimus-project.eu/).

Repository is organized as set of demos with increasing complexity. Each folder contains a ROS 2 package launching the demo. README.md of each demo contains description with expected results and instruction how to launch the demo.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Repository is organized as set of demos with increasing complexity. Each folder contains a ROS 2 package launching the demo. README.md of each demo contains description with expected results and instruction how to launch the demo.
The repository is organized as set of demos with increasing complexity.
Each folder contains a ROS 2 package launching the demo.
The `README.md` file of each demo contains a detailed description of the expected results and instruction on how to launch the demo.

@Dyymon Dyymon mentioned this pull request Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants