Skip to content
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

Update ros2-over-ethernet.rst to fix issue with ip adresses of companion and ardupilot #6590

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nido009
Copy link

@nido009 nido009 commented Jan 31, 2025

changed IP-Adresse of Ardupilot and Companion-Computer to match the config-parameters in the table below. stumbled upon this when trying to find out if the DDS_IP-Parameters shall have the IP of the Companion or the Ardupilot itself.

changed IP-Adresse of Ardupilot and Companion-Computer
@Hwurzburg Hwurzburg requested a review from Ryanf55 January 31, 2025 20:23
@rmackay9
Copy link
Contributor

I'm probably misunderstanding but by default, AP autopilots should have the IP address of 192.168.144.14 I think. In general if we can stick with the defaults that should make it easier for users to setup the system

@Ryanf55
Copy link
Contributor

Ryanf55 commented Feb 1, 2025

Question - the defaults look to be set here in AP_Networking_config.h.

The values supplied in the wiki was just a hypothetical from the addresses my DHCP server gave them.

// Default Static IP Address: 192.168.144.14
#ifndef AP_NETWORKING_DEFAULT_STATIC_IP_ADDR
#define AP_NETWORKING_DEFAULT_STATIC_IP_ADDR "192.168.144.14"
#endif

Maybe we should say :

* An autopilot has the IP address ``192.168.144.14``
* A Computer running the MicroROS agent has the IP address ``192.168.144.6``
* The MicroROS agent is running on port ``2019``

This would match the rest of the ethernet wikis that have now been written.

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