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.

Sign up for GitHub

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

Jump to bottom

Improve integration of expander module #18

Closed
6 of 7 tasks
falkoschindler opened this issue Nov 8, 2023 · 0 comments · Fixed by #74 or #85 · May be fixed by #97
Closed
6 of 7 tasks

Improve integration of expander module #18

falkoschindler opened this issue Nov 8, 2023 · 0 comments · Fixed by #74 or #85 · May be fixed by #97
Assignees
@falkoschindler @JensOgorek
Labels
enhancement New feature or request
Milestone
0.6.0

Comments

@falkoschindler
Copy link
Collaborator

falkoschindler commented Nov 8, 2023 •
edited by JensOgorek
Loading

  • When instantiating an expander module, the execution should wait until the expander has sent a Ready message.
  • When instantiating a proxy module, the execution should wait until the expander has sent a first broadcast message (see Unknown Property "level" #66).
  • Later we could add constructor parameters for skipping the wait to save time when working with many expanders.
  • New methods like expander.await_ready() or proxy.await_broadcast() allow awaiting many expanders at once.
  • A new last_message_age property can be used to define safety rules that stop the machine on connection loss.
  • Expressions for reading properties need to be evaluated at runtime, not at compile time (see PR Introduce property expression to avoid evaluating properties too early #51).
  • Some kind of heart beat message should make sure a lost connection to the expander is detected.
@falkoschindler falkoschindler added the enhancement New feature or request label Nov 8, 2023
@falkoschindler falkoschindler self-assigned this Nov 8, 2023
@falkoschindler falkoschindler mentioned this issue Apr 17, 2024
Closed
@falkoschindler falkoschindler mentioned this issue Jun 28, 2024
Merged
3 tasks
@falkoschindler falkoschindler mentioned this issue Sep 5, 2024
Open
@JensOgorek JensOgorek self-assigned this Sep 26, 2024
@JensOgorek JensOgorek mentioned this issue Sep 30, 2024
Merged
10 tasks
@JensOgorek JensOgorek linked a pull request Sep 30, 2024 that will close this issue
Expander improvements I #74
Merged
10 tasks
@JensOgorek JensOgorek mentioned this issue Nov 4, 2024
Merged
5 tasks
@JensOgorek JensOgorek linked a pull request Nov 4, 2024 that will close this issue
Expander improvements II #85
Merged
5 tasks
@JensOgorek JensOgorek added this to the 0.6.0 milestone Nov 6, 2024
@JensOgorek JensOgorek mentioned this issue Nov 15, 2024
Open
6 tasks
@JensOgorek JensOgorek linked a pull request Nov 18, 2024 that will close this issue
Proxy defaults #97
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@falkoschindler falkoschindler

@JensOgorek JensOgorek

Labels
enhancement New feature or request
Projects
None yet
Milestone
0.6.0
2 participants
@falkoschindler @JensOgorek