More robust initialization against a delay of info#7
Open
Cryst4L9527 wants to merge 1 commit intoHaiderAbasi:masterfrom
Open
More robust initialization against a delay of info#7Cryst4L9527 wants to merge 1 commit intoHaiderAbasi:masterfrom
Cryst4L9527 wants to merge 1 commit intoHaiderAbasi:masterfrom
Conversation
Author
|
Maybe you can just confirm this? Just a small change of codes for better robustness, it won't change the main functions of this program. Thank you~ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi! I'm a researcher on bug detection and software robustness focusing on robotic applications. It's our new idea to design a language-irrelevant algorithm, so we are trying to check and fix some python-based programs to prove our algorithm. Since your program is one of the best opensource codes in python for the ROS system, we did a test and found some crashes:
Analysis:
There is a chance some of the messages such as
/BotCamera/image_rawmay have a delay in the real situation, also even in the simulation of gazebo, which is the reason of the crashes occurred when we tested.Solution:
Therefore, I just change the logic of maze_solving: check first if a real bot_view and sat_view are received , then continue to do the rest things.
Hoping you can agree this merge.