Skip to content

Conversation

@sansha
Copy link
Contributor

@sansha sansha commented Jul 25, 2025

Backporting #25231 into the 1.16 release. Thank you!

Solved Problem

When ... I found that ...

Fixes #{Github issue ID}

Solution

  • Add ... for ...
  • Refactor ...

Changelog Entry

For release notes:

Feature/Bugfix XYZ
New parameter: XYZ_Z
Documentation: Need to clarify page ... / done, read docs.px4.io/...

Alternatives

We could also ...

Test coverage

Context

Related links, screenshot before/after, video

* Read XRCE_DDS IP from ENV

* just remove the -h flag from the launch command

* add 127.0.0.1 as default IP

* add default value for IP
@sansha
Copy link
Contributor Author

sansha commented Jul 25, 2025

@beniaminopozzan the failing build seems unrelated, is there any way to retrigger this?

@sansha sansha changed the title Backport of fix: let UXRCE DDS agent IP to be set via parameter in SITL (#25231) [Backport 1.16] of fix: let UXRCE DDS agent IP to be set via parameter in SITL (#25231) Jul 28, 2025
@sansha sansha force-pushed the release/1.16-backport-#25231 branch from b9826cf to 1261caa Compare July 28, 2025 15:08
@sansha
Copy link
Contributor Author

sansha commented Jul 28, 2025

@beniaminopozzan I just removed the empty commits to make the history clean, saw CI failing in the same place also on other PRs. I think this is ready whenever you have time :) Thanks!

@mrpollo
Copy link
Contributor

mrpollo commented Jul 31, 2025

Thanks for the contribution. This seems worthwhile, although a bit late for the release.

Regarding the CI test, I made this PR to fix the issue. I'll make sure we bring this in as soon as we get it passing #25344.

@sansha
Copy link
Contributor Author

sansha commented Aug 1, 2025

@mrpollo awesome, thank you so much! let me know when I can help with anything

@mrpollo mrpollo merged commit 61079bd into PX4:release/1.16 Aug 6, 2025
59 of 62 checks passed
@DronecodeBot
Copy link

This pull request has been mentioned on Discussion Forum for PX4, Pixhawk, QGroundControl, MAVSDK, MAVLink. There might be relevant details there:

https://discuss.px4.io/t/px4-dev-call-aug-06-2025-team-sync-and-community-q-a/46826/1

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.

3 participants