Skip to content
This repository has been archived by the owner on Sep 24, 2024. It is now read-only.

added webots #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

added webots #1

wants to merge 4 commits into from

Conversation

delihus
Copy link

@delihus delihus commented Dec 28, 2022

No description provided.


> **Prerequisites**
>
> The `compose.sim.webots.yaml` file uses NVIDIA Container Runtime. Make sure you have NVIDIA GPU and the [NVIDIA Container Toolkit](https://docs.nvidia.com/datacenter/cloud-native/container-toolkit/install-guide.html) installed.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it possible to run webots without nvidia runtime? For ignition-gazebo simulation we have two configs (the one without nvidia runtime is really really slow though).

Copy link
Author

Choose a reason for hiding this comment

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

Added in readme running without nvidia.

volumes:
- ./maps:/maps
environment:
- DDS_CONFIG
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if DDS_CONFIG is necessary, and if it is, it should be also included in other containers.

Copy link
Author

Choose a reason for hiding this comment

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

Added in every service

enable_interactive_mode: false

# General Parameters
use_scan_matching: false
Copy link
Contributor

Choose a reason for hiding this comment

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

still not fixed? :D It would be nice to add a comment that this parameter is changed, and a general one at the beginning of the file, explaining why it is necessary to have a separate config. Also a link to the issue will be nice, so that we can keep track of it, and remove this config once it is solved.

Copy link
Author

Choose a reason for hiding this comment

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

Added the comment at the beginning of the file.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants