open-rmf / open-rmf/crossflow

Prevent async channels from sending streams after the service finishes

Open
#29 0 comments 0 reactions 1 assignee View on GitHub

@mxgrey is already working on this.

Since Oct 26, 2024.

Dominant language
Rust
Stars
72
Forks
16
PR merge metrics
No merged PRs in 30d

Description

Currently it is possible for a user to implement an async service that sends its async channel outside of the function scope so that the channel can outlive the function scope. Then the channel could continue to be used to transmit new stream signals after the service has terminated.

This violates bevy_impulse's contract that services cannot produce stream data after their final output has been delivered.

We should put a mechanism into the async channel that allows us to close it once we have received the final output for the service call that the channel was created for.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.