Bugfix: Recorder discovery does not restart after being stopped #1894
Conversation
Repeated calls to start_discovery should not start additional topics discovery tasks without stop_discovery calls Signed-off-by: Øystein Sture <[email protected]>
|
The can_record_again_after_stop is failing because it records one more message than expected. If discovery now works between the record calls, it makes sense that it records something else as well, e.g., rosout? |
There was a problem hiding this comment.
@oysstu thank you for PR and attempt to fix a wrong behavior.
The naming is hard! It seems the root cause of the issue is in unclear naming fo the variable stop_discovery_.
The logic would be more readable and much easy to understand if stop_discovery_ would be renamed to the discovery_runing_.
Could you please do such rename in the code and change conditions accordingly?
MichaelOrlov
changed the title
Signed-off-by: Øystein Sture <[email protected]>
|
I've made the change, that should clarify the logic a lot. Edit: seemed to pass tests this time |
|
Pulls: #1894
|
|
https://github.com/Mergifyio backport jazzy |
✅ Backports have been created
|
* Invert start discovery condition Repeated calls to start_discovery should not start additional topics discovery tasks without stop_discovery calls Signed-off-by: Øystein Sture <[email protected]> * Rename stop_discovery to discovery_running and change logic accordingly Signed-off-by: Øystein Sture <[email protected]> --------- Signed-off-by: Øystein Sture <[email protected]> (cherry picked from commit b483541)
This is a bug fix for a situation when the discovery in Recorder does not restart after being stopped.
Repeated calls to start_discovery should not start additional topics discovery tasks without stop_discovery calls.