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

Don't publish LRAUV state while paused #91

Merged
merged 2 commits into from
Nov 24, 2021
Merged

Conversation

chapulina
Copy link
Contributor

TethysComm has been publishing the latest state even while simulation is paused.

Not publishing while paused helps with debugging, because if we're echoing messages or checking console logs, we can step through them and analyze the latest data slowly. This may also help the controller, I'm not sure what it's doing with all these repeated messages that it is receiving.

One downside of this is that some sensor data may be received between the last publish and the pause, and it won't be published until unpause. But I'm not sure this causes any actual issues.


I also went ahead and deleted an unused variable, and moved another one down closer to where it is used.

Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina chapulina self-assigned this Nov 23, 2021
Copy link
Member

@arjo129 arjo129 left a comment

Choose a reason for hiding this comment

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

Thanks, tested on my computer against the LRAUV-application and it works. This is a great QOL improvement.

@mabelzhang
Copy link
Collaborator

mabelzhang commented Nov 24, 2021

The only thing I can think of is the time syncing on the MBARI side. That is done with the /stats message now, so I THINK it is okay. (Two timestamps are being written to the Slate, and I don't know how often that's written. One is the Ignition time from the State message, and the other is a timeExtWriter_ that writes the elapsed time. I don't remember whether that picks up the pause from a higher controller.)

We made particular effort to make sure time sync was still in perfect step even after pausing for a long time.

To be prudent, someone should test the yoyo mission manually while pausing it a few times (short duration, and long duration, like > 10 seconds) in the middle, and make sure it doesn't make sudden turns or weird movements.

P.S. To pause console logs though, Ctrl+S is handy. Then Ctrl+Q to unpause.

@arjo129
Copy link
Member

arjo129 commented Nov 24, 2021

Yea I tested that behavior seems to work as expected.

@chapulina chapulina merged commit d6c305f into main Nov 24, 2021
@chapulina chapulina deleted the chapulina/no_comm_paused branch November 24, 2021 03:33
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