Skip to content

add multicast support #11

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

shiv-tyagi
Copy link

This adds multicast support to the mavesp8266 firmware. The user can select between unicast and multicast mode using the web interface and specify the multicast ip and the port. The multicast functionality works only when the esp module is connected to an access point using the STA interface. When doing multicast, the esp modules can communicate with each other on the specified multicast ip and port.
The default multicast ip and port are 239.255.145.50 and 14550 respectively.
The user can connect mavproxy to all the vehicles in multicast mode using the command mavproxy.py --master mcast: --console --map (if the multicast ip and port are set to default and the gcs is also connected to the same access point).
image

Copy link

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we change the labels for unicast-specific items in the web interface to be prefixed? e.g. "Unicast xyzzy"

Comment on lines +44 to +45
#define CAST_MODE_UNI 0
#define CAST_MODE_MULTI 1

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we really change the default?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed on discord, we are not changing any defaults here. Thanks.

@peterbarker
Copy link

... should also bump the version up.

@shiv-tyagi
Copy link
Author

shiv-tyagi commented Aug 26, 2023

Should we change the labels for unicast-specific items in the web interface to be prefixed? e.g. "Unicast xyzzy"

I don't think there are many such fields. Just the Host port and client port. I have changed those.

... should also bump the version up.

Done. Thanks.

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