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

added Auto signal #681

Merged
merged 8 commits into from
Nov 22, 2023
Merged

added Auto signal #681

merged 8 commits into from
Nov 22, 2023

Conversation

UmangSharmaMobis
Copy link
Contributor

No description provided.

Signed-off-by: Umang Sharma <[email protected]>
Signed-off-by: Umang Sharma <[email protected]>
description: Auto Power Optimization Flag When set to 'true',
the system enables automatic power optimization,
dynamically adjusting the power optimization level based on runtime conditions
or features managed by the OEM. When set to 'false',
Copy link
Collaborator

Choose a reason for hiding this comment

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

There is a trailing blank on this line causing CI to fail

@erikbosch erikbosch added Status:Rework Committer must refactor or address comments Status:Review Please review/discuss contents labels Nov 1, 2023
@erikbosch
Copy link
Collaborator

For this one to be considered for merging, please fix the errors reported from Continuous Integration

Signed-off-by: Umang Sharma <[email protected]>
Signed-off-by: Umang Sharma <[email protected]>
Signed-off-by: Umang Sharma <[email protected]>
Signed-off-by: Umang Sharma <[email protected]>
Signed-off-by: Umang Sharma <[email protected]>
@UmangSharmaMobis
Copy link
Contributor Author

I was on leave without access to my system, I've now successfully resolved the errors reported from Continuous Integration.

@@ -108,3 +108,12 @@ PowerOptimizeLevel:
A higher number indicates more aggressive power optimization.
Level 0 indicates that all functionality is enabled, no power optimization enabled.
Level 10 indicates most aggressive power optimization mode, only essential functionality enabled.

isAutoPowerOptimize:
Copy link
Collaborator

@erikbosch erikbosch Nov 21, 2023

Choose a reason for hiding this comment

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

I would prefer capital "I", i.e. IsAutoPowerOptimize as all other VSS signals starts with capital.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated

Signed-off-by: Umang Sharma <[email protected]>
@erikbosch erikbosch removed the Status:Rework Committer must refactor or address comments label Nov 21, 2023
@erikbosch
Copy link
Collaborator

Meeting notes: merge

@erikbosch erikbosch merged commit fe5ff86 into COVESA:master Nov 22, 2023
4 checks passed
erikbosch pushed a commit to boschglobal/vehicle_signal_specification that referenced this pull request Dec 4, 2023
Add IsAutoPowerOptimize

Signed-off-by: Umang Sharma <[email protected]>
erikbosch pushed a commit that referenced this pull request Dec 4, 2023
Add IsAutoPowerOptimize

Signed-off-by: Umang Sharma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status:Review Please review/discuss contents
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants