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

Introspection throughput calculation with building blocks #402

Open
marthtz opened this issue Nov 30, 2020 · 2 comments
Open

Introspection throughput calculation with building blocks #402

marthtz opened this issue Nov 30, 2020 · 2 comments
Labels
enhancement New feature refactoring Refactor code without adding features
Milestone

Comments

@marthtz
Copy link
Contributor

marthtz commented Nov 30, 2020

Brief feature description

Introspection throughput calculation must be adapted to work with building blocks. In adidtion, the throughput calculation should have less impact on the system as it used to.

Detailed information

Add introspection throughput calculation with building blocks. However, we don't want pay for throughput calculation in every publisher just because somebody could be interested.

For example:

  • In ROS you can do it with the tools by doing something like ros2 topic /myTopic bw. If a gateway needs this information the job would be there. For the introspection it would be easy to have this as an extra functionality, maybe also per topic and maybe as separate application?
  • Should we start throughput calc only when the introspection is started/registered at RouDi? This could apply to all introspection related data (mempool info etc.)
@marthtz marthtz added refactoring Refactor code without adding features enhancement New feature labels Nov 30, 2020
@mossmaurice mossmaurice added this to the Prio 2 milestone Dec 1, 2020
@mossmaurice
Copy link
Contributor

Had a chat with @dkroenke about this issue today. We could even consider re-writing the introspection completely e.g. in Python or Rust. Downside would be more dependencies when getting iceoryx up running. Any thoughts?

@elBoberido
Copy link
Member

Had a chat with @dkroenke about this issue today. We could even consider re-writing the introspection completely e.g. in Python or Rust. Downside would be more dependencies when getting iceoryx up running. Any thoughts?

I'm all for rust, but since it doesn't yet support all platforms we need, Python might be the better choice here. Nevertheless, we need Python bindings!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature refactoring Refactor code without adding features
Projects
None yet
Development

No branches or pull requests

3 participants