-
Notifications
You must be signed in to change notification settings - Fork 172
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
Introduce TimeInUse signal for ElectricMotor #725
Conversation
Looks good to me, will bring it up on the VSS meeting today. |
58a5043
to
bf0a3a5
Compare
Thank you @erikbosch |
MoM:
|
a19ee5b
to
34b0e11
Compare
datatype: float | ||
type: sensor | ||
description: Accumulated time during engine lifetime when the vehicule state's is READY. | ||
comment: Vehicles may define their READY state. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we see a need to add an iSReady
signal as well (in this file and/or on vehicle level)? I do not know how common the term is - and if so if there is a common definition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't have any use for it at the moment, and if it were to be added, I think it would have to be part of another Pull Request.
MoM:
|
Let's start from the beginning. What we want: What is the trigger: A question was about the charging time. If you want this signal to remove an ambiguity, we can add a ‘TimeInCharge’ signal based on the ‘Charging.IsCharging’ signal. |
34b0e11
to
56be973
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This signal is now well defined. At last meeting there were some discussions on use-cases for it. I have no problem to see potential use-cases, it nothing else to collect statistics on time-in-use vs. wear and problems with vehicles. That off course does not mean that all parties interested in VSS may find it useful, but that is the same for many other signals.
MoM:
|
Signed-off-by: Jonathan Force <[email protected]>
56be973
to
215cdda
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
MoM: Merge |
Description
It would be interesting to know the engine hours for electric motor.