irobot/master-events-executor features #87
Conversation
This file contains 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
* replace spin_until_future_complete with spin in component_manager_isolated.hpp spin_until_future_complete() is more inefficient as it needs to check the state of the future and the timeout after every work iteration Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * fix uncrustify error Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com>
Signed-off-by: Chen Lihui <lihui.chen@sony.com>
* use universal reference to support rvalue. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
* initial attempt, problems with ament_gen_version_h Signed-off-by: Shane Loretz <sloretz@osrfoundation.org> * remove blank line Signed-off-by: Shane Loretz <sloretz@osrfoundation.org>
Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
* Add RMW listener APIs Signed-off-by: Alberto Soragna <asoragna@irobot.com> * split long log into two lines Signed-off-by: Alberto Soragna <asoragna@irobot.com> * Remove use_previous_event Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * Do not set gc listener callback Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * significant refactor to make callbacks type safe Signed-off-by: William Woodall <william@osrfoundation.org> * Add on_ready callback to waitables, add clear_* functions, add unit-tests Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * add mutex to set and clear listener APIs Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * allow to set ipc sub callback from regular subscription Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * fix minor failures Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * fix typos and errors in comments Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * fix comments Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * expose qos listener APIs from pub and sub; add unit-tests Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * add qos event unit-test for invalid callbacks Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * Use QoS depth to limit callbacks count Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * fix ipc-subscription Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * Rename CallbackMessageT -> ROSMessageType Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * Set callbacks to Actions Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * changes from upstream Signed-off-by: William Woodall <william@osrfoundation.org> * Unset callback on entities destruction Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * fix rebase errors Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * fix unit-tests Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * Add GuardCondition on trigger callback Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * Add tests for new GuardCondition APIs Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * Fix windows CI Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * Action unset callbacks only if were set Signed-off-by: Mauro Passerino <mpasserino@irobot.com> * add missing include rcl event callback include directive Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com> * typos Signed-off-by: William Woodall <william@osrfoundation.org> * remove listener reference Signed-off-by: William Woodall <william@osrfoundation.org> * remove references to listener and move a method closer to impl Signed-off-by: William Woodall <william@osrfoundation.org> * cpplint Signed-off-by: William Woodall <william@osrfoundation.org> * Fix qos history check in subscription_intra_process.hpp Co-authored-by: Mauro Passerino <mpasserino@irobot.com> Co-authored-by: William Woodall <william@osrfoundation.org> Co-authored-by: Alberto Soragna <alberto.soragna@gmail.com>
small improvements to node_main.cpp.in
* spin_all with a zero timeout. Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Alberto Soragna <alberto.soragna@gmail.com>
* Use a single call to collect all CallbackGroup pointers. Believe it or not, the taking and releasing of mutex_ within the CallbackGroup class shows up in profiles. However, when collecting entities we really don't need to take it and drop it between each of subscriptions, services, clients, timers, and waitables. Just take it once at the beginning, collect everything, and then return, which removes a lot of unnecessary work with that mutex. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org> * Pass shared_ptr as constref in the MemoryStrategy class. That way, in the case that the callee doesn't need to take a reference, we avoid creating and destroying a shared_ptr class. This reduces the overhead in using these functions, and seems to work fine in the default case. If a user wants to subclass the MemoryStrategy class, then they can explicitly create a shared_ptr copy by using the copy constructor. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org> * Use constref shared_ptr when iterating. Believe it or not, the creation and destruction of the shared_ptr class itself shows up in profiles in these loops. Since we don't need to hold onto a reference while iterating (the class is already doing that), just use constref wherever we can which drastically reduces the overhead. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* time_until_trigger returns max time if timer is cancelled Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
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.
This PR is a placeholder for current works on
EventsExecutoron_trigger_callbackto guard conditions.on_reset_callbackAPIs to wake up when a timer is reset.