Skip to content

DeepWater Exploration - DWE OS Extension#132

Merged
joaoantoniocardoso merged 1 commit intobluerobotics:masterfrom
DeepWaterExploration:master
May 14, 2025
Merged

DeepWater Exploration - DWE OS Extension#132
joaoantoniocardoso merged 1 commit intobluerobotics:masterfrom
DeepWaterExploration:master

Conversation

@brandonhs
Copy link
Copy Markdown
Contributor

Request for the DWE OS extension to be added to the BlueRobotics extension repository.

There may be some things involving the setup that require changing. Let me know if any additional information is needed.

A few weeks ago, settings saving appeared to be working for most users, but recently it seems that named volumes may no longer be supported by BlueOS? It seems like some testing will need to be done prior to merging this commit. Some of our users have reported problems with the extension, specifically regarding saving settings and occasionally the docker container losing connection, but that appears to be a unique case.

For now, I will flag this as a draft pull request while we conduct testing on both sides.

@brandonhs brandonhs marked this pull request as ready for review May 12, 2025 22:41
@brandonhs
Copy link
Copy Markdown
Contributor Author

brandonhs commented May 12, 2025

We have tested internally and it seems like the latest docker image is working well. The extension appears to be missing a background image, and the README overflows over the content width for the extension description.

We also plan to replace the existing UI with our new designed standard. At the same time, it may make sense to remove the streaming component of our extension altogether to reduce confusion.

@joaoantoniocardoso joaoantoniocardoso self-requested a review May 13, 2025 19:17
@joaoantoniocardoso
Copy link
Copy Markdown
Member

Also tested here with your camera on BlueOS 1.4.0, all good!

@joaoantoniocardoso joaoantoniocardoso merged commit d1bfe38 into bluerobotics:master May 14, 2025
2 checks passed
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.

2 participants