forked from Robotic-Decision-Making-Lab/blue
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Build for Jazzy with mavros from source #1
Closed
Closed
Changes from 5 commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
216f36a
Create "robot" dev container.
amarburg 0374f6f
Build mavros from source.
amarburg f19a49b
Switch CI distro to "jazzy"
amarburg b90ee87
Run pre-commit.
amarburg 08fc821
Switch to Gazebo "harmonic"
amarburg 8793a86
Remove robot dir to clean MR
amarburg 453f124
Use user "ubuntu" provided by upstream image.
amarburg 5d50646
Minor updates to comments.
amarburg 0e9c698
Remove extraneous debug "ls"
amarburg e9f4a4e
Move install_geographiclib_datasets below mavros deps install.
amarburg 3afc1e7
CI workflow uses image from this repo
amarburg 5f15589
Use github.repository, not repository_id
amarburg 933d2e6
Update a few more users "blue" -> "ubuntu"
amarburg 119ef0b
Make colcon play nice with venvs:
amarburg 1f81b7c
Explicitly install libgstreamer1.0-dev and libgstreamer-plugins-base1…
amarburg 99c5f26
Don't modify /usr/bin/colcon, use an alias instead.
amarburg File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,29 @@ | ||
# | ||
# Dockerfile for *-robot development container | ||
# | ||
FROM ghcr.io/robotic-decision-making-lab/blue:jazzy-robot | ||
|
||
# Install ROS dependencies | ||
# This is done in a previous stage, but we include it again here in case anyone wants to | ||
# add new dependencies during development | ||
ENV USERNAME=blue | ||
ENV USER_WORKSPACE=/home/$USERNAME/ws_blue | ||
WORKDIR $USER_WORKSPACE | ||
|
||
COPY --chown=$USER_UID:$USER_GID . src/blue | ||
RUN sudo apt-get -q update \ | ||
&& sudo apt-get -q -y upgrade \ | ||
&& rosdep update \ | ||
&& rosdep install -y --from-paths src --ignore-src --rosdistro ${ROS_DISTRO} --skip-keys="gz-transport12 gz-sim7 gz-math7 gz-msgs9 gz-plugin2" \ | ||
&& sudo apt-get autoremove -y \ | ||
&& sudo apt-get clean -y \ | ||
&& sudo rm -rf /var/lib/apt/lists/* | ||
|
||
# Install debugging/linting Python packages | ||
RUN /home/$USERNAME/.venv/blue/bin/python3 -m pip install \ | ||
pre-commit \ | ||
mypy | ||
|
||
# Disable the setuputils installation warning | ||
# This prevents us from needing to pin the setuputils version (which doesn't always work) | ||
ENV PYTHONWARNINGS="ignore" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,39 @@ | ||
{ | ||
"name": "Robot Dev Container", | ||
"dockerFile": "Dockerfile", | ||
"context": "../..", | ||
"workspaceMount": "source=${localWorkspaceFolder},target=/home/blue/ws_blue/src/blue,type=bind", | ||
"workspaceFolder": "/home/blue/ws_blue/src/blue", | ||
"remoteUser": "blue", | ||
"runArgs": [ | ||
"--network=host", | ||
"--cap-add=SYS_PTRACE", | ||
"--security-opt=seccomp:unconfined", | ||
"--security-opt=apparmor:unconfined", | ||
"--volume=/dev:/dev", | ||
"--privileged", | ||
"--volume=/run/user/1000:/run/user/1000" | ||
], | ||
"containerEnv": { | ||
"DISPLAY": "${localEnv:DISPLAY}", | ||
"WAYLAND_DISPLAY": "${localEnv:WAYLAND_DISPLAY}", | ||
"XDG_RUNTIME_DIR": "${localEnv:XDG_RUNTIME_DIR}", | ||
"PULSE_SERVER": "${localEnv:PULSE_SERVER}" | ||
}, | ||
"customizations": { | ||
"vscode": { | ||
"extensions": [ | ||
"ms-azuretools.vscode-docker", | ||
"ms-python.python", | ||
"njpwerner.autodocstring", | ||
"redhat.vscode-xml", | ||
"redhat.vscode-yaml", | ||
"smilerobotics.urdf", | ||
"esbenp.prettier-vscode", | ||
"charliermarsh.ruff", | ||
"josetr.cmake-language-support-vscode", | ||
"unifiedjs.vscode-mdx" | ||
] | ||
} | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
diff --git a/pymavlink/generator/mavgen.py b/pymavlink/generator/mavgen.py | ||
index ea62c961..35353184 100755 | ||
--- a/pymavlink/generator/mavgen.py | ||
+++ b/pymavlink/generator/mavgen.py | ||
@@ -23,8 +23,8 @@ General process: | ||
''' | ||
|
||
from __future__ import print_function | ||
-from future import standard_library | ||
-standard_library.install_aliases() | ||
+#from future import standard_library | ||
+#standard_library.install_aliases() | ||
from builtins import object | ||
import os | ||
import re |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,4 +8,5 @@ | |
!blue_demos | ||
!blue.repos | ||
!.docker/entrypoints | ||
!.docker/patches | ||
!requirements-build.txt |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may run into permissions issues here when trying to use the devcontainer. The blue user likely won't belong to the same group as your personal user on your local system (set to 1000 by default). My solution has been to go along with the ubuntu user standardization.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, I hadn't considered the permission ramifications. So you are using the user
ubuntu
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah. This was motivated by discussions in cloud-images!2005129, which make it seem like the Ubuntu devs are attempting to standardize the
ubuntu
user. You could still delete theubuntu
user to regain access to UID 1000, but I opted against that.