Rename Docker image and minor fix for build script

[mabelzhang]

• Rename Docker image to mbari_wec to match DockerHub repo
• Minor fix for build script:
  Do not tag with extra suffix (_nvidia | _no_nvidia) if just building nvidia image.
  Previously, it mistakenly tagged the nvidia base image too, resulting in an extra tag like nvidia_opengl_ubuntu22_nvidia.
  The suffix was only meant for the main buoy image.
• Add instructions to README for pushing to DockerHub

[github-actions]

PR Preview Action 68cd47d
Preview removed because the pull request was closed.
2023-04-14 18:42 UTC

[mabelzhang]

Okay, I've tested this. You should be able to follow the README to build with the new Docker image name (mbari_wec) and push to DockerHub.

Feel free to try out the instructions and push to DockerHub. We're not going to push a final image anyway until the logging feature is in.

I've only tried the run_simulation.bash, which just brings up the buoy simulation with ros2 launch buoy_gazebo mbari_wec.launch.py. Feel free to test other things in there, e.g. a tutorial, other features, to make sure it really works.

Note that by default, it builds the NVIDIA version. You can use --no-nvidia, but the graphics might be slow depending on what kind of graphics card you have.

[quarkytale]

Yet to try pushing to DockerHub but other things work fine

[mabelzhang]

Thanks!