[refactor] Modular tasks
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 27
- Forks
- 30
- PR merge metrics
- No merged PRs in 30d
Description
Currently, rmf_task::Request is used to define a bunch of requests (loop, delivery, clean, chargebattery) that are later converted into rmf_fleet_adapter::Task within the fleet adapter. Each rmf_fleet_adapter::Task is composed of a string of phases present in src/rmf_fleet_adapter/phases. The definitions of these requests/tasks are rigid and the process of introducing new tasks is fairly involved.
The proposal is to move some of the Phases into the public API in rmf_task. These would serve as building blocks to describe tasks. Hence requests would be described as a collection of these phases. Each phase/block would also inherit from an error handling class that dictates how errors during the phase should be resolved. Similarly for when the phase is cancelled. Each phase would also have invariant_duration and estimate_finish methods to allow the task planner to estimate the duration of the phase and the state of the agent at the end of it.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by tracing how rmf_task::Request is converted into rmf_fleet_adapter::Task, then read the existing phase definitions under src/rmf_fleet_adapter/phases. Determine which phases can become public rmf_task building blocks and how error handling, cancellation, invariant_duration, and estimate_finish should be represented. Done means the task model supports composing requests from these phases without the current rigid conversion process.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- backend-api-design
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100