-
Notifications
You must be signed in to change notification settings - Fork 60
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
Event based lift / door logic #320
Conversation
Signed-off-by: Luca Della Vedova <[email protected]>
Signed-off-by: Luca Della Vedova <[email protected]>
Signed-off-by: Luca Della Vedova <[email protected]>
Signed-off-by: Luca Della Vedova <[email protected]>
This should be ready for review. It shouldn't be too controversial and these should be strictly improvements in the latency of the system, what will be more controversial instead is the changes on the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tested this out, no problems at all. It should be a pure upgrade to how messages are currently handled, and fully backwards compatible 👍
Signed-off-by: Luca Della Vedova <[email protected]> Signed-off-by: Arjo Chakravarty <[email protected]>
This reverts commit 5e58af9. Signed-off-by: Luca Della Vedova <[email protected]>
This reverts commit 5e58af9. Signed-off-by: Luca Della Vedova <[email protected]>
New feature implementation
Implemented feature
Part of open-rmf/rmf_simulation#114.
Implementation description
This PR performs a few small changes in how we interact with infrastructure: