Skip to content

added docker support#33

Open
taku0625 wants to merge 9 commits intormurai0610:mainfrom
taku0625:feature/docker
Open

added docker support#33
taku0625 wants to merge 9 commits intormurai0610:mainfrom
taku0625:feature/docker

Conversation

@taku0625
Copy link
Copy Markdown

@taku0625 taku0625 commented Mar 2, 2025

Thank you for your nice work.
I added docker support. Please note docker.md before building the docker image.

@taku0625 taku0625 changed the title Feature/docker add docker support Mar 2, 2025
@taku0625 taku0625 changed the title add docker support added docker support Mar 2, 2025
@rmurai0610
Copy link
Copy Markdown
Owner

Hi, thank you for making this PR! We'll try running it locally when we have time to setup a docker environment.

@BashMocha
Copy link
Copy Markdown

Hey,

I'm getting this error when I run the build.sh script:

validating /home/xyz/mast3r-slam/docker-compose.yaml: services.mast3r-slam Additional property gpus is not allowed.

My Docker Compose version is v2.28.1.

Couldn't find any way to solve this. Any ideas?

@sebastienmascha
Copy link
Copy Markdown

Hey,

I'm getting this error when I run the build.sh script:

validating /home/xyz/mast3r-slam/docker-compose.yaml: services.mast3r-slam Additional property gpus is not allowed.

My Docker Compose version is v2.28.1.

Couldn't find any way to solve this. Any ideas?

This is a runtime issue because of the docker-compose.yaml configuration. You need to remove gpus: all from docker/docker-compose.yaml. You can instead use:

    deploy:
      resources:
        reservations:
          devices:
            - driver: nvidia
              count: all
              capabilities: [gpu]

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.

5 participants