Skip to content
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

Missing simple RX application features #58

Open
roland-sipos opened this issue Apr 27, 2023 · 1 comment
Open

Missing simple RX application features #58

roland-sipos opened this issue Apr 27, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@roland-sipos
Copy link
Member

List of features to be implemented for the simple RX receiver (fdaq equivalent):

@roland-sipos roland-sipos added the enhancement New feature or request label Apr 27, 2023
@gkokkoro
Copy link
Contributor

  • The universal stream id (det + crate + slot +strm) works but it’s indecipherable - need something more structured
  • Detailed per-stream stats are essential (seq_id jumps, unexpected_packet size, etc)
  • The number of lookup maps should be minimised (now there are 3) to avoid unnecessary lookups - worth looking into having a single map with a struct as value (stuct holding all the counters per stream)
  • Too much using namespace …
  • The m_buf->data_len does include ethernet and udp header (+1 mysterious byte I haven’t managed to understand). The length of those headers needs to be subtracted before comparison

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants