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

Try to stop everything before setting config list #35

Merged
merged 7 commits into from
Mar 21, 2023

Conversation

slaible
Copy link
Collaborator

@slaible slaible commented Mar 7, 2023

Problem:
The bridge tries to set the config parameters when starting. When the localization (or something else) is already running at the beginning, setting config parameters at the same time is not possible in the Locator, and the bridge crashes. To fix this, you have to use, for example, the aXessor.

Solution:
Try to stop everything before setting the parameters.

Better solution:
Instead of just trying to stop everything, it would be better to check the state of the Locator before. But information at the ClientControlMode interface is only available at state changes. Is there another API for checking the states, @syyuen, @HWasserfuhr ?

Signed-off-by: Stefan Laible <stefan.laible@de.bosch.com>
Signed-off-by: Stefan Laible <stefan.laible@de.bosch.com>
Signed-off-by: Stefan Laible <stefan.laible@de.bosch.com>
Signed-off-by: Stefan Laible <stefan.laible@de.bosch.com>
@kfabian
Copy link
Collaborator

kfabian commented Mar 8, 2023

action-ros-ci@v0.3 has removed support for galactic, so I guess this will break the galactic build when merged. ros-tooling/action-ros-ci@f3f9f5a

Signed-off-by: Stefan Laible <stefan.laible@de.bosch.com>
Signed-off-by: Stefan Laible <stefan.laible@de.bosch.com>
@slaible
Copy link
Collaborator Author

slaible commented Mar 9, 2023

action-ros-ci@v0.3 has removed support for galactic, so I guess this will break the galactic build when merged. ros-tooling/action-ros-ci@f3f9f5a

@kfabian, thanks for the hint. I changed the version to v0.2. I thought using a new version would maybe solve Build action (utils): Galactic, but it is still failing. The build fails when the pep257 test is processing the python launch file of the utils package. Running the same test locally, the test succeeds.

Signed-off-by: Stefan Laible <stefan.laible@de.bosch.com>
@slaible slaible merged commit 06babcc into main Mar 21, 2023
@slaible slaible deleted the bugfix/not-in-required-state branch March 21, 2023 11:06
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