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

Add ability to see the reason/source for several statuses/commands #67

Closed
Tracked by #132
otterdahl opened this issue Sep 11, 2019 · 3 comments
Closed
Tracked by #132
Milestone

Comments

@otterdahl
Copy link
Contributor

For instance, reason for plan change, dark mode, yellow flash, and so forth.

The reason for changing of signal plan could be timetable, command, etc.

@otterdahl otterdahl added this to the 1.0.16 milestone Sep 11, 2019
@emiltin
Copy link
Contributor

emiltin commented Oct 29, 2021

i prefer 'source' over 'reason'.

@otterdahl
Copy link
Contributor Author

The suggestion is to add a "return value", called "source" to the following statuses:

  • S0007 Controller switched on
  • S0008 Manual control
  • S0009 Fixed time control
  • S0010 Isolated control
  • S0011 Yellow flash
  • S0012 All red
  • S0014 Current time plan
  • S0015 Current traffic situation

"Source" is a text field which contain the reason for the new status, e.g. "operator panel", "calendar clock", "control block", "forced" etc. The exact format is not specifed.

Since we just add a return value to each status, this addition is considered to be backwards compatible. The supervision system will not recieve this value unless it is requested.

otterdahl added a commit that referenced this issue Nov 4, 2021
"Source" contain the reason for the new status,
e.g. "operator panel", "calendar clock", "control block", "forced" etc..

Affects the following statuses:

    S0007 Controller switched on
    S0008 Manual control
    S0009 Fixed time control
    S0010 Isolated control
    S0011 Yellow flash
    S0012 All red
    S0014 Current time plan
    S0015 Current traffic situation

Discussion in #67
@otterdahl
Copy link
Contributor Author

On second though, use a predefined list of reasons for the new status

@otterdahl otterdahl mentioned this issue Nov 4, 2021
30 tasks
otterdahl added a commit that referenced this issue Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants