Skip to content

Add Discovery Server setting in the UI #70

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: main
Choose a base branch
from

Conversation

MMarcus95
Copy link

This PR adds the possibility to discover and visualize topics in Plotjuggler when using Discovery Server setting.

The UI has been changed as follows

Screenshot from 2025-01-24 12-44-20

How to use

As in the current behavior, when the Select DDS message window appears, by default a domain participant in the Current DomainId domain is created, which uses Simple Discovery.

To allow Plotjuggler to discover topics when using Discovery Server setting you need to:

  • set the server ip in the server IP field
  • set the server port in the server port field
  • enable the Discovery Server setting by clicking on the button OFF

Once the Discovery Server setting is enable, you will see:

  • the button changing to ON
  • the Change Domain button becoming gray, meaning that when the Discovery Server setting is ON you cannot use Simple Discovery, avoiding to mix incompatible discovery settings

Screenshot from 2025-01-24 12-44-58

When the Discovery Server setting is turned off by clicking the ON button, a domain participant is created in the Current DomainId domain using Simple Discovery and the possibility use Simple Discovery and to change the domain is restored.

Notice that server IP and server port need to be set before starting the discovery server, so when the button is in OFF state.

Signed-off-by: Marco Marchitto <marco.marchitto@iit.it>
Signed-off-by: Marco Marchitto <marco.marchitto@iit.it>
@MMarcus95 MMarcus95 marked this pull request as ready for review January 24, 2025 13:37
@juanlofer-eprosima
Copy link
Contributor

@MMarcus95 Thanks for your contribution! We will be allocating some time in the next days to review your PR.

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