-
Notifications
You must be signed in to change notification settings - Fork 1
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
Ardusub install #20
Ardusub install #20
Conversation
@rakeshv24 Pinging you that the image |
Will test it soon! |
@woensug-choi I tested |
@rakeshv24 Lovely! 👍 |
@rakeshv24 new Also pushed
|
If we were to use dave/extras/repos/mavros.jazzy.repos Line 5 in b77235d
2.8.0 (that of the latest release). We may need to change it later to official release version.
|
We need to fix and merge this PR as well @woensug-choi |
@rakeshv24 You mean fixing branch names in dockerfile and install script right? Does the functionality you intended using ardusub work? or are we postponding ardusub functions to ver 1.1? |
@woensug-choi yes that is what I meant (changing branch names in docker file and installation scripts). Also, the functionality I intended works fine. In this version, the bluerov models will be launched with ardusub process. In later releases, we will target advanced functionality. |
PR for new baseline docker image to resolve #19
What's been added
/opt/ardupilot_dave
/opt/mavros
Image is being built from
dockertest
branchros2
anddockertest
. The latest update of this branch is merged todockertest
which triggered the Github CI build using self-hosted runner (a local linux machine in my lab). It may take upto 24 hours for full ARM and AMD build.dockertest
branch. It will showfailing
until action is done. CheckActions
tab on top.To-do before merge to
ros2
ardusub_install
to$BRANCH
atdave/.docker/jazzy.amd64.dockerfile
Line 57 in f91e807
dave/.docker/jazzy.amd64.dockerfile
Line 61 in f91e807
dave/.docker/jazzy.arm64v8.dockerfile
Line 108 in f91e807
dave/.docker/jazzy.arm64v8.dockerfile
Line 112 in f91e807
dave/extras/mavros-ubuntu-install.sh
Lines 21 to 22 in f91e807