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

Introduce TimeInUse signal for ElectricMotor #725

Merged
merged 1 commit into from
Mar 12, 2024

Conversation

JonathanForce
Copy link
Contributor

Description

It would be interesting to know the engine hours for electric motor.

@erikbosch erikbosch added Scope:Minor A change that is not major and not trivial. Status:New An issue/PR that not yet have been discussed/announced at a VSS meeting Status:Meeting Intended to be discussed at next VSS-project meeting labels Feb 20, 2024
@erikbosch
Copy link
Collaborator

Looks good to me, will bring it up on the VSS meeting today.
You need to add a sign-off to the commit, for example with git commit -s --amend and then a force push

See https://covesa.global/contribute

@JonathanForce
Copy link
Contributor Author

Thank you @erikbosch

@erikbosch
Copy link
Collaborator

MoM:

  • Adnan: Possible remove the term engine, call it "motor hours", "active hours", "running hours", "hours in use"
  • Possibly "Time in use"
  • Adnan: Do use EV use rotation as criteria?
  • Erik: When do we want to increase timer, only when motor rotating, or rather some status ("ready")
  • Please review

@JonathanForce JonathanForce force-pushed the electric_hours_in_use branch 2 times, most recently from a19ee5b to 34b0e11 Compare February 21, 2024 20:09
@JonathanForce JonathanForce changed the title Introduce EngineHours signal for ElectricMotor Introduce TimeInUse signal for ElectricMotor Feb 21, 2024
datatype: float
type: sensor
description: Accumulated time during engine lifetime when the vehicule state's is READY.
comment: Vehicles may define their READY state.
Copy link
Collaborator

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.

Copy link
Contributor Author

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.

@erikbosch erikbosch removed the Status:New An issue/PR that not yet have been discussed/announced at a VSS meeting label Feb 27, 2024
@erikbosch
Copy link
Collaborator

MoM:

  • Jonathon: For us READY state is that if gas pedal is pressed vehicle will move. But definition can vary
  • Daniel: Unclear from when it is measured
  • J: Total time in use
  • Erik: Shall Park be considered READY?
  • Nick: In my vauxhall when moved out of Park
  • S: Also charging is part of vehicle lifetime
  • Please continue review/discuss, lets continue discussion on next meeting

@JonathanForce
Copy link
Contributor Author

MoM:

  • Jonathon: For us READY state is that if gas pedal is pressed vehicle will move. But definition can vary
  • Daniel: Unclear from when it is measured
  • J: Total time in use
  • Erik: Shall Park be considered READY?
  • Nick: In my vauxhall when moved out of Park
  • S: Also charging is part of vehicle lifetime
  • Please continue review/discuss, lets continue discussion on next meeting

Let's start from the beginning.

What we want:
For electric vehicles, we want to determine a degree of wear on parts based on how long the vehicle has been in use.

What is the trigger:
We want this time to increase as soon as the vehicle is ON and the transmission is not in the 'Park' position.
So, the condition is 'Transmission.SelectedGear' is not '126=Park’.

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.

@JonathanForce JonathanForce force-pushed the electric_hours_in_use branch from 34b0e11 to 56be973 Compare March 4, 2024 15:23
Copy link
Collaborator

@erikbosch erikbosch left a 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.

@erikbosch
Copy link
Collaborator

MoM:

  • Stefan: Terminal50 (Klemme 50) could be used - if you turn key you have ACC on first step, then ON, think it is standard. Also others exist like Terminal30 (permanent).
  • Erik: Partially different criteria compared to using gear!=park
  • Stefan: Also electrical components degrade, not only motor
  • Pierre: What is use case?
  • Jonathan: Do not want to increment timer if just listening at radio
  • J: At BRP we use hydrofoils and other non-standard vehicle types
  • S: You want to know when high voltage connection is closed/engaged.
  • E: Do not know how EV handle if it is "Park" for a long while, does it turn off (disengage) the EV-drive.
  • E: An alternative would be to have it less general, like "READY" in a previous commit version
  • P: Would be fine to leave it to OEM decide.
  • P: But could good to be document why it useful:
  • E: AP Jonathan; Change to "READY", add rationale for why signal is useful as comment
  • J: That was my first proposal, but comment for current solution last week
  • J: I can update but I will not participate in meetings onward
  • E: AP Jonathan to update
  • Merge discussion next week

@JonathanForce JonathanForce force-pushed the electric_hours_in_use branch from 56be973 to 215cdda Compare March 7, 2024 14:42
Copy link
Collaborator

@erikbosch erikbosch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@erikbosch
Copy link
Collaborator

MoM: Merge

@erikbosch erikbosch merged commit 4ff82b6 into COVESA:master Mar 12, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope:Minor A change that is not major and not trivial. Status:Meeting Intended to be discussed at next VSS-project meeting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants