Support lifecycle-managed timers in rclcpp_lifecycle::LifecycleNode
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 30/100
Direzione di ricerca
Inizia con LifecycleNode::create_timer() e create_wall_timer(), quindi esamina il comportamento delle transizioni del ciclo di vita descritto qui e la discussione sul design in PR #2261. Risolvi con i maintainer le questioni aperte sull’API e sullo stato della transizione; il lavoro è completato quando sono disponibili un’API per i timer gestita dal ciclo di vita, concordata con i maintainer, e test che coprano gli stati inattivo e attivo, le transizioni ripetute, la creazione mentre è attivo e le transizioni non riuscite.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
rclcpp_lifecycle::LifecycleNode does not provide an API for managing timer callback execution based on lifecycle state.
Currently, LifecycleNode::create_timer() and create_wall_timer() create regular rclcpp timers whose callback execution is not managed by lifecycle transitions.
As a result, if an executor is spinning, timer callbacks may run even when the node is Unconfigured or Inactive.
This is not a claim that the existing timer APIs are incorrect.
The issue is that LifecycleNode does not provide an API for opting into lifecycle-managed timer behavior.
This issue proposes the following behavior for lifecycle-managed timers:
| Lifecycle state or transition | Proposed timer behavior |
|---|---|
Unconfigured |
Do not start executing new timer callbacks |
Inactive |
Do not start executing new timer callbacks |
Successful transition to Active |
Start or resume callback execution for timers that have already been created |
Active |
Allow lifecycle-managed timer callbacks to execute |
| Successful deactivation | cancel() the timer; callbacks already in flight may complete |
| Successful cleanup / shutdown | Prevent new timer callbacks from starting and clean up the timer |
The start behavior for a timer created while the node is already Active, and the behavior when a lifecycle transition fails, are left as open questions below.
Motivation
Applications may need to control periodic timer-based work according to lifecycle state.
However, because LifecycleNode does not provide a lifecycle-managed timer option, each application must implement this control separately.
The ROS 2 Managed Nodes design describes Inactive as a state in which the node is not performing any processing.
The lifecycle_talker demo also uses a regular timer and includes a comment that a future version will add a timer following the same lifecycle management as a lifecycle publisher. This comment provides historical context that lifecycle-managed timers have been considered before.
Inspection of the LifecycleNode implementations in Rolling and Jazzy found that neither branch provides a separate lifecycle-managed timer API.
On Jazzy, I confirmed that callbacks from a regular create_wall_timer() execute while the node is both Unconfigured and Inactive.
This result demonstrates the current behavior of regular timers; it does not imply that the existing API violates an explicitly documented lifecycle contract.
Design / Implementation Considerations
There are three possible API directions to consider.
Add a new API
- Add separate APIs for creating lifecycle-managed timers.
Change the existing APIs
- Change the default behavior of
LifecycleNode::create_timer()andcreate_wall_timer()so that they create lifecycle-managed timers. - Add an option to the existing APIs for selecting managed or unmanaged behavior.
For now, I propose adding separate APIs.
Changing the default behavior of the existing APIs could alter timer behavior in Inactive without a compile error for existing applications that use auto or rclcpp::TimerBase::SharedPtr.
Adding an option would preserve existing calls, but the same function would provide different lifecycle behavior, requiring readers to inspect the option at each call site to determine whether a timer is lifecycle-managed.
Separate APIs would preserve existing application behavior while making the distinction between regular and lifecycle-managed timers explicit at each call site.
For example, the following APIs could be considered:
create_lifecycle_timer(...)
create_lifecycle_wall_timer(...)
To assess the feasibility of the proposed behavior, I tested a downstream implementation that calls cancel() and reset() on timers in response to lifecycle transitions.
The Jazzy-based tests confirmed that new callbacks did not start while the node was Inactive, callbacks executed while it was Active, and timer callback execution resumed across repeated activate/deactivate cycles.
This validation only establishes the feasibility of the proposed lifecycle behavior.
The actual upstream API names, signatures, return types, and internal management approach remain open for discussion with the maintainers.
PR #2261 proposed separate APIs for lifecycle-managed timers and integration with lifecycle transitions, but it was closed without being merged.
The review raised design questions about whether callbacks should execute before activation under different autostart settings and how a timer created while the node is already Active should behave.
This issue does not request adopting the implementation from #2261 as-is.
Its purpose is to clarify behavior based on lifecycle state and timer creation timing, and to revisit the API design for lifecycle-managed timers.
I would appreciate maintainer guidance on the following:
- Is lifecycle-managed timer support still desired for Rolling?
- Which direction would be preferable: adding separate APIs, changing the default behavior of the existing APIs, or adding an option to the existing APIs?
- If a lifecycle-managed timer is created while the node is already
Active, should it start immediately? - How should a managed timer behave when a lifecycle transition fails?
Additional Information
Lifecycle-managed timer support for rclcpp_lifecycle::LifecycleNode in Rolling
- Lingua principale
- C++
- Stelle
- 805
- Fork
- 564
- Merge medio
- 1g 22h
- PR unite (30g)
- 21
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di ros2/rclcpp
-
good first issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
Component: GLib
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
brave/brave-browser#59300 ·
-
Mute ydb/tests/functional/dstool/test_canonical_requests.py.Test.test_group_take_snapshot in main Apertaai_reviewed
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
ydb-platform/ydb#53974 · 3 commenti ·